Sweep: Set up GHA that runs tsc on push/pull to default branch #5

Closed
opened 2026-02-15 19:15:36 -05:00 by yindo · 2 comments
Owner

Originally created by @kevinlu1248 on GitHub (Jul 26, 2023).

Originally created by @kevinlu1248 on GitHub (Jul 26, 2023).
yindo added the sweep label 2026-02-15 19:15:36 -05:00
yindo closed this issue 2026-02-15 19:15:36 -05:00
Author
Owner

@sweep-ai-deprecated[bot] commented on GitHub (Jul 26, 2023):

Here's the PR! https://github.com/run-llama/LlamaIndexTS/pull/40.

💎 Sweep Pro: I used GPT-4 to create this ticket. You have 1034 GPT-4 tickets left.


Step 1: 🔍 Code Search

I found the following snippets in your repository. I will now analyze these snippets and come up with a plan.

Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description.

https://github.com/run-llama/LlamaIndexTS/blob/fe3b1f4f50ab5837a0ce2d85ebb2771170a25671/packages/core/src/llm/LLM.ts#L1-L100

https://github.com/run-llama/LlamaIndexTS/blob/fe3b1f4f50ab5837a0ce2d85ebb2771170a25671/packages/core/src/ResponseSynthesizer.ts#L1-L79

https://github.com/run-llama/LlamaIndexTS/blob/fe3b1f4f50ab5837a0ce2d85ebb2771170a25671/packages/core/src/ChatEngine.ts#L1-L70

https://github.com/run-llama/LlamaIndexTS/blob/fe3b1f4f50ab5837a0ce2d85ebb2771170a25671/packages/core/src/tests/utility/mockOpenAI.ts#L1-L40

https://github.com/run-llama/LlamaIndexTS/blob/fe3b1f4f50ab5837a0ce2d85ebb2771170a25671/apps/docs/docs/api/modules.md#L188-L313


Step 2: 🧐 Snippet Analysis

From looking through the relevant snippets, I decided to make the following modifications:

File Path Proposed Changes
.github/workflows/lint_on_push_or_pull.yml Create this file to define the new GitHub Actions workflow. The file should include the triggers (on push or pull request to the default branch) and the job that runs the pnpm run lint command.

Step 3: 📝 Planning

I have created a plan for writing the pull request. I am now working my plan and coding the required changes to address this issue. Here is the planned pull request:

Add GitHub Actions workflow for linting on push/pull
sweep/add-gha-workflow

Description

This PR adds a new GitHub Actions workflow file to the repository. The workflow is triggered on every push or pull request to the default branch and runs the pnpm run lint command to ensure code quality.

Summary of Changes

  • Created a new GitHub Actions workflow file, lint_on_push_or_pull.yml, in the .github/workflows directory.
  • Configured the workflow to trigger on every push or pull request to the default branch.
  • Added a job to the workflow that runs the pnpm run lint command.

Step 4: ⌨️ Coding

I have finished coding the issue. I am now reviewing it for completeness.


Step 5: 🔁 Code Review

Success! 🚀


I'm a bot that handles simple bugs and feature requests but I might make mistakes. Please be kind!
Join Our Discord

@sweep-ai-deprecated[bot] commented on GitHub (Jul 26, 2023): ## Here's the PR! [https://github.com/run-llama/LlamaIndexTS/pull/40](https://github.com/run-llama/LlamaIndexTS/pull/40). 💎 Sweep Pro: I used GPT-4 to create this ticket. You have 1034 GPT-4 tickets left. * Install Sweep Configs: [Pull Request](https://github.com/run-llama/LlamaIndexTS/pull/36) --- ## Step 1: 🔍 Code Search I found the following snippets in your repository. I will now analyze these snippets and come up with a plan. <details> <summary>Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description.</summary> https://github.com/run-llama/LlamaIndexTS/blob/fe3b1f4f50ab5837a0ce2d85ebb2771170a25671/packages/core/src/llm/LLM.ts#L1-L100 https://github.com/run-llama/LlamaIndexTS/blob/fe3b1f4f50ab5837a0ce2d85ebb2771170a25671/packages/core/src/ResponseSynthesizer.ts#L1-L79 https://github.com/run-llama/LlamaIndexTS/blob/fe3b1f4f50ab5837a0ce2d85ebb2771170a25671/packages/core/src/ChatEngine.ts#L1-L70 https://github.com/run-llama/LlamaIndexTS/blob/fe3b1f4f50ab5837a0ce2d85ebb2771170a25671/packages/core/src/tests/utility/mockOpenAI.ts#L1-L40 https://github.com/run-llama/LlamaIndexTS/blob/fe3b1f4f50ab5837a0ce2d85ebb2771170a25671/apps/docs/docs/api/modules.md#L188-L313 </details> --- ## Step 2: 🧐 Snippet Analysis From looking through the relevant snippets, I decided to make the following modifications: | File Path | Proposed Changes | |:---------------------------------------------|:----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------| | `.github/workflows/lint_on_push_or_pull.yml` | Create this file to define the new GitHub Actions workflow. The file should include the triggers (on push or pull request to the default branch) and the job that runs the `pnpm run lint` command. | --- ## Step 3: 📝 Planning I have created a plan for writing the pull request. I am now working my plan and coding the required changes to address this issue. Here is the planned pull request: **Add GitHub Actions workflow for linting on push/pull** `sweep/add-gha-workflow` >## Description >This PR adds a new GitHub Actions workflow file to the repository. The workflow is triggered on every push or pull request to the default branch and runs the `pnpm run lint` command to ensure code quality. > >## Summary of Changes >- Created a new GitHub Actions workflow file, `lint_on_push_or_pull.yml`, in the `.github/workflows` directory. >- Configured the workflow to trigger on every push or pull request to the default branch. >- Added a job to the workflow that runs the `pnpm run lint` command. --- ## Step 4: ⌨️ Coding I have finished coding the issue. I am now reviewing it for completeness. --- ## Step 5: 🔁 Code Review Success! 🚀 --- I'm a bot that handles simple bugs and feature requests but I might make mistakes. Please be kind! <sup>[Join Our Discord](https://discord.com/invite/sweep-ai)
Author
Owner

@kevinlu1248 commented on GitHub (Jul 26, 2023):

sweep: Retry but instead of tsc just do pnpm run lint

@kevinlu1248 commented on GitHub (Jul 26, 2023): sweep: Retry but instead of tsc just do `pnpm run lint`
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: run-llama/LlamaIndexTS#5