chore: remove cleanup step from integration tests workflow #2551
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
- Add pnpm cache step using actions/cache@v4 for node_modules and pnpm store - Remove redundant pnpm cache from setup-node step - Add plugin dist cache to prevent unnecessary rebuilds Similar to the caching strategy in smoke-tests.yml, this change should significantly reduce workflow execution time by caching both dependencies and build artifacts. Co-Authored-By: Sayo <[email protected]>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
⚙️ Control Options:
|
|
lets test for a while |
wtfsayo
approved these changes
Jan 20, 2025
devin-ai-integration
bot
changed the title
mgunnin
added a commit
to mgunnin/eliza-agent
that referenced
this pull request
Jan 20, 2025
* upstream/develop: test configuration and tests for client-lens (elizaOS#2534) chore(attempt): optimize pnpm cache configuration (elizaOS#2556) feat: add a way to create/store/restore agents in the filesystem (elizaOS#2389) chore: optimize pnpm cache configuration (elizaOS#2555) chore: remove cleanup step from integration tests workflow (elizaOS#2553) feat: Add more actions to Abstract Plugin (elizaOS#2531) feat: add anthropic image provider for vision (elizaOS#2524) feat: improve integrationtests workflow caching (elizaOS#2551) feat: Updated READ.me file with pre-requisites to enable telegram bot (elizaOS#2547) feat(plugin-devin): implement client-agnostic Devin plugin (elizaOS#2549) use generateObject handle undefined env variable feat:add plugin-lightning (elizaOS#2429)
0xrubusdata
pushed a commit
to 0xrubusdata/eliza
that referenced
this pull request
Jan 21, 2025
- Add pnpm cache step using actions/cache@v4 for node_modules and pnpm store - Remove redundant pnpm cache from setup-node step - Add plugin dist cache to prevent unnecessary rebuilds Similar to the caching strategy in smoke-tests.yml, this change should significantly reduce workflow execution time by caching both dependencies and build artifacts. Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: Sayo <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Remove cleanup step from integration tests workflow
This PR removes the cleanup step from the integration tests workflow as requested. The cleanup step was running
pnpm cleanwhich is no longer needed in the workflow.Changes
Testing
The changes can be verified by:
Link to Devin run: https://app.devin.ai/sessions/ff5037c60d2f46e38d68f36060e13a2d