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

chore: Cache swift packages - WPB-9913 #1991

Open
wants to merge 66 commits into
base: develop
Choose a base branch
from

Conversation

KaterinaWire
Copy link
Contributor

@KaterinaWire KaterinaWire commented Oct 1, 2024

Issue

As we adding more and more swift packages the CI should cache (them similar to Carthage) instead of downloading all packages every time.

Solution

Add steps to the workflow (similar to the Carthage) to cash and Bootstrap swift packages.

Screenshot 2024-10-02 at 09 19 01

Testing


Checklist

  • Title contains a reference JIRA issue number like [WPB-XXX].
  • Description is filled and free of optional paragraphs.
  • Adds/updates automated tests.

UI accessibility checklist

If your PR includes UI changes, please utilize this checklist:

  • Make sure you use the API for UI elements that support large fonts.
  • All colors are taken from WireDesign.ColorTheme or constructed using WireDesign.BaseColorPalette.
  • New UI elements have Accessibility strings for VoiceOver.

@echoes-hq echoes-hq bot added the echoes: throughput/ci-maintenance Changes we need to do to keep CI healthy and fast label Oct 1, 2024
Copy link
Contributor

github-actions bot commented Oct 1, 2024

Test Results

327 tests   327 ✅  29s ⏱️
 52 suites    0 💤
  5 files      0 ❌

Results for commit 3e8c522.

♻️ This comment has been updated with latest results.

@KaterinaWire KaterinaWire changed the title chore: Cache swift packages (Temp) chore: Cache swift packages - WPB-9913 Oct 2, 2024
@KaterinaWire KaterinaWire marked this pull request as ready for review October 2, 2024 07:25
id: cache-scripts-package
with:
path: scripts/.build
key: ${{ runner.os }}-swiftpm-scripts-${{ hashFiles('scripts/Package.resolved') }}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

the key here is macOS-swiftpm-scripts-

id: cache-sourceryPlugin-package
with:
path: SourceryPlugin/.build
key: ${{ runner.os }}-swiftpm-sourcery-${{ hashFiles('SourceryPlugin/Package.resolved') }}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

the key here is macOS-swiftpm-sourcery-

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT MERGE echoes: throughput/ci-maintenance Changes we need to do to keep CI healthy and fast
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants