Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Sweep GHA Fix] The GitHub Actions run failed with... #337

Conversation

sweep-ai[bot]
Copy link
Contributor

@sweep-ai sweep-ai bot commented Jan 10, 2024

Description

This pull request includes the following changes:

  • Updated the e2e.yml workflow file to include a new environment variable PLAYWRIGHT_BROWSERS_PATH and redirect the output of the pnpm exec playwright test command to a file called error_logs.txt.
  • Added a new workflow file lint.yml for linting the code.
  • Updated the test.yml workflow file to use the actions/checkout@v2 and actions/setup-node@v2 actions, and changed the pnpm install and pnpm run test commands to use npm instead of pnpm.
  • Made changes to the LLM.ts file, including removing the import of MessageContent from ../ChatEngine, adding new interfaces LLMChatParamsBase, LLMChatParamsStreaming, LLMChatParamsNonStreaming, LLMCompletionParamsBase, LLMCompletionParamsStreaming, and LLMCompletionParamsNonStreaming, and updating the chat and complete methods.
  • Updated the OpenAI.ts file to extend the BaseLLM class and make changes to the chat and complete methods.
  • Updated the LlamaDeuce.ts file to extend the BaseLLM class and remove the hasStreaming property.
  • Updated the Anthropic.ts file to extend the BaseLLM class and remove the hasStreaming property.
  • Updated the Portkey.ts file to extend the BaseLLM class and remove the hasStreaming property.
  • Updated the MistralAI.ts file to remove the hasStreaming property.
  • Updated the CallbackManager.test.ts file to import the CallbackManager class from the correct location.
  • Updated the Embedding.test.ts file to import the necessary classes and interfaces from the correct location.
  • Updated the README-template.md file to fix the links to the LlamaIndex GitHub repository.
  • Updated the main.py file to import the FastAPI and CORSMiddleware classes from the correct location.
  • Updated the README-template.md file in the streaming/fastapi template to fix the link to the LlamaIndex GitHub repository.

Copy link

vercel bot commented Jan 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
llama-index-ts-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 10, 2024 0:43am

Copy link
Contributor Author

sweep-ai bot commented Jan 10, 2024

Apply Sweep Rules to your PR?

  • Apply: All new business logic should have corresponding unit tests.
  • Apply: Refactor large functions to be more modular.
  • Apply: Add docstrings to all functions and file headers.

Copy link
Contributor Author

sweep-ai bot commented Jan 10, 2024

Sweeping

Fixing PR: track the progress here.

I'm currently fixing this PR to address the following:

[Sweep GHA Fix] The GitHub Actions run failed with the following error logs:

The command:
Run actions/setup-node@v2
yielded the following error:
##[error]The 'pnpm store path' command failed with exit code: 1

Here are the logs:
Attempting to download 14.x...
Acquiring 14.21.3 - x64 from https://github.com/actions/node-versions/releases/download/14.21.3-4202774076/node-14.21.3-linux-x64.tar.gz
Extracting ...
[command]/usr/bin/tar xz --strip 1 --warning=no-unknown-keyword --overwrite -C /home/runner/work/_temp/6add05a1-b78c-4585-9dea-d957d907cac5 -f /home/runner/work/_temp/d4794d10-782e-4460-b4c7-8aad33afe44c
Adding to the cache ...
Done
[command]/home/runner/setup-pnpm/node_modules/.bin/pnpm store path
ERROR: This version of pnpm requires at least Node.js v16.14
The current version of Node.js is v14.21.3
Visit https://r.pnpm.io/comp to see the list of past pnpm versions with respective Node.js version support.

[!CAUTION]

An error has occurred: 422 {"message": "Validation Failed", "errors": [{"resource": "PullRequest", "code": "custom", "message": "No commits between sweep/sweep_gha_fix_the_github_actions_run_fai_17 and sweep/sweep_gha_fix_the_github_actions_run_fai_20"}], "documentation_url": "https://docs.github.com/rest/pulls/pulls#create-a-pull-request"} (tracking ID: e13f7ac7bf)

Copy link
Contributor Author

sweep-ai bot commented Jan 10, 2024

Sweeping

Fixing PR: track the progress here.

I'm currently fixing this PR to address the following:

[Sweep GHA Fix] The GitHub Actions run failed with the following error logs:

The command:
Run actions/setup-node@v2
yielded the following error:
##[error]The 'pnpm store path' command failed with exit code: 1

Here are the logs:
Attempting to download 14.x...
Acquiring 14.21.3 - x64 from https://github.com/actions/node-versions/releases/download/14.21.3-4202774076/node-14.21.3-linux-x64.tar.gz
Extracting ...
[command]/usr/bin/tar xz --strip 1 --warning=no-unknown-keyword --overwrite -C /home/runner/work/_temp/8802c113-87c7-4460-b3e8-dd6d1105f08f -f /home/runner/work/_temp/223a6935-5d3c-4b86-a80e-7d00f1ccf906
Adding to the cache ...
Done
[command]/home/runner/setup-pnpm/node_modules/.bin/pnpm store path
ERROR: This version of pnpm requires at least Node.js v16.14
The current version of Node.js is v14.21.3
Visit https://r.pnpm.io/comp to see the list of past pnpm versions with respective Node.js version support.

Making changes according to plan... (step 3/3)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant