search

danger / swift

⚠️ Stop saying "you forgot to …" in code review
https://danger.systems/swift/
MIT License
1.04k stars 136 forks source link

Cache dependencies on CI #467

Closed 417-72KI closed 2 years ago

417-72KI commented 2 years ago

Sometimes CI fails due to fail fetch dependencies from GitHub. Caching fetched files may suppress wasting time.

↓ has been duplicated with #414 😅 Additionally, macos-latest host runner will use macos-11. So I ~~unify running CI on macOS host to macos-11 replace macos-latest with macos-10.15.~~ (Installing danger-js with Xcode 11.7 in macos-11 failed due to Homebrew😅.)

ref: https://github.com/actions/virtual-environments/blob/main/images/macos/macos-11-Readme.md#installed-sdks

417-72KI commented 2 years ago

@f-meloni Partially duplicated PR (#414) has been merged and conflicted, so I resolved it. May I merge this by myself if there is no problem?

f-meloni commented 2 years ago

@f-meloni Partially duplicated PR (#414) has been merged and conflicted, so I resolved it. May I merge this by myself if there is no problem?

sure :)