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

Modernize github workflows #3711

Merged
merged 3 commits into from
Aug 22, 2024
Merged

Conversation

WorldSEnder
Copy link
Member

@WorldSEnder WorldSEnder commented Aug 22, 2024

Description

This mainly fixes some inconsistencies in style, outdated or wrong comments and action versions

  • replace Legit-Labs/action-download-artifact with actions/download-artifact
  • fix Swatinem/rust-cache arguments and be more aggressive about cache reusage. We are near or over the 10GB limit. Meanwhile rust-cache has gotten better and better of only caching dependencies and final build outputs. Use shared-key to group things into only a few categories. Note that environment flags will still lead to different caches being used.
  • Fix benchmark transformations and benchmarking in general by supplying the chrome binary path
  • Expand feature soundness lints to include the other packages exposed to users and fix the comment of how to generate the commands locally with the newest version of cargo hack
  • Wording and capitalization in comments

All in all, it looks a bit messy, but this is supposed to take care of quite a bit of cruft.

Checklist

  • I have reviewed my own code
  • I have added tests

github-actions[bot]
github-actions bot previously approved these changes Aug 22, 2024
Copy link

github-actions bot commented Aug 22, 2024

Visit the preview URL for this PR (updated for commit f2652ad):

https://yew-rs-api--pr3711-github-dl-artifacts-ugq27tvs.web.app

(expires Thu, 29 Aug 2024 00:48:25 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

this mainly fixes some inconsistencies in style, outdated or wrong comments and action version
- replace Legit-Labs/action-download-artifact with actions/download-artifact
- fix Swatinem/rust-cache arguments
- fix benchmark transformations
- expand feature soundness lints
- wording and capitalization in comments
github-actions[bot]
github-actions bot previously approved these changes Aug 22, 2024
@WorldSEnder WorldSEnder requested a review from ranile August 22, 2024 00:53
@ranile ranile merged commit 3871418 into yewstack:master Aug 22, 2024
23 checks passed
@WorldSEnder WorldSEnder added the A-ci Area: The continuous integration label Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-ci Area: The continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants