Skip to content

Add caching for faster dependency installation in Smoke test#4526

Closed
Anshukumar123975 wants to merge 7 commits intosugarlabs:masterfrom
Anshukumar123975:master
Closed

Add caching for faster dependency installation in Smoke test#4526
Anshukumar123975 wants to merge 7 commits intosugarlabs:masterfrom
Anshukumar123975:master

Conversation

@Anshukumar123975
Copy link
Contributor

Closes #4525

@github-actions
Copy link
Contributor

✅ All Jest tests passed! This PR is ready to merge.

@Anshukumar123975
Copy link
Contributor Author

Hey @walterbender, please review my PR.
Let me know if any improvements are needed.
Thank you!

@omsuneri
Copy link
Member

@Anshukumar123975 refer #4525 (comment)
also try to write a proper commit message from next time as "update ----" is not a good commit message also on long run the commit message will only stay with us
also try to add a proper description to the PR as its better and easier for reviewing as "closes #---" is not a good description for this you can refer our contributions guide.

@Anshukumar123975
Copy link
Contributor Author

@omsuneri Got it! I'll be more mindful of writing meaningful commit messages and providing clear PR descriptions in the future.

@github-actions
Copy link
Contributor

✅ All Jest tests passed! This PR is ready to merge.

@github-actions
Copy link
Contributor

✅ All Jest tests passed! This PR is ready to merge.

@github-actions
Copy link
Contributor

✅ All Jest tests passed! This PR is ready to merge.

@github-actions
Copy link
Contributor

✅ All Jest tests passed! This PR is ready to merge.

@github-actions
Copy link
Contributor

✅ All Jest tests passed! This PR is ready to merge.

@github-actions
Copy link
Contributor

✅ All Jest tests passed! This PR is ready to merge.

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.

[CI]Add caching for faster dependency installation in Smoke test

2 participants