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

Add caching for sbt dependencies and build artifacts (Issue #342) #344

Closed
wants to merge 2 commits into from
Closed

Conversation

samadpls
Copy link

(Issue #342 )
This pull request adds caching steps to the CI workflow to reduce the time and bandwidth needed for each job. It uses the actions/cache@v2 action to cache the sbt dependencies and build artifacts based on the hash of the sbt files and the operating system. This can improve the performance and reliability of the CI workflow.

- name: Cache sbt dependencies
uses: actions/cache@v2
with:
path: |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, these items are already cached by other actions

- name: Cache sbt build artifacts
uses: actions/cache@v2
with:
path: |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These could be helpful but they don't really reduce the build time by much (~2m), I have an internal version which I'll push

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your reply, Alex. I appreciate your feedback on the pull request and it's great to hear that you have an internal version that can further improve the build time. In that case, would you like me to close this pull request for now? Please let me know your thoughts

@AlexITC
Copy link
Contributor

AlexITC commented Mar 18, 2023

Superseded by 729c98a

@AlexITC AlexITC closed this Mar 18, 2023
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.

2 participants