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

CI: switch from quickinstall to install-action #2021

Merged
merged 22 commits into from
Mar 14, 2023
Merged

Conversation

indietyp
Copy link
Member

@indietyp indietyp commented Feb 8, 2023

🌟 What is the purpose of this PR?

Currently, we use quickinstall to install all dependencies required, but (especially the lint job) jobs have been failing spuriously. This tries to fix the issue by installing the big brother of cargo-quickinstall: cargo-binstall, which is from the same organization, but more actively maintained and active.

binstall falls back to the quickinstall repo of precompiled binaries if available.

🔗 Related links

📹 Demo

@github-actions github-actions bot added the area/infra Relates to version control, CI, CD or IaC (area) label Feb 8, 2023
@indietyp
Copy link
Member Author

indietyp commented Feb 8, 2023

Nice side-effect: install tools now only takes 14s instead of 51s (for error-stack).

Copy link
Member

@TimDiekmann TimDiekmann left a comment

Choose a reason for hiding this comment

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

Worth a try 🎉

However, do we want to try out the install action instead? It fallsback to cargo-bininstall.

Nice side-effect: install tools now only takes 14s instead of 51s (for error-stack).

Well, I doubt, that cargo install bininstall is faster than cargo install quickinstall

.github/workflows/rust.yml Outdated Show resolved Hide resolved
@indietyp
Copy link
Member Author

indietyp commented Feb 8, 2023

Well, I doubt, that cargo install bininstall is faster than cargo install quickinstall

It isn't but we're circumventing cargo install binstall by using the installation instructions of the README.md and downloading a pre-built binary.

@indietyp indietyp changed the title CI: switch from quickinstall to binstall CI: switch from quickinstall to install-action Feb 8, 2023
@indietyp
Copy link
Member Author

indietyp commented Feb 8, 2023

deer lint still magically failed, opened issue for binstall

(This does not yet work, need to wait for upstream)
@indietyp
Copy link
Member Author

Same here, but I need this to be ready (not draft), as otherwise, I am unable to run our CI suite

@TimDiekmann
Copy link
Member

Triage: Upstream issue is resolved but no release was filed yet.

@indietyp
Copy link
Member Author

Triage: new release has been cut upstream, will run CI a couple of time to ensure no regressions.

@indietyp
Copy link
Member Author

CI ran five times with the new version, each time with ~14 tool installations, therefore I think it is safe to say the previous bug has been fixed.

TimDiekmann
TimDiekmann previously approved these changes Mar 14, 2023
Copy link
Member

@TimDiekmann TimDiekmann left a comment

Choose a reason for hiding this comment

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

Sounds good!

.github/workflows/rust.yml Outdated Show resolved Hide resolved
cargo quickinstall cargo-hack --version 0.5.26 || cargo install --version 0.5.26 cargo-hack
uses: taiki-e/install-action@v2
with:
tool: cargo-make@0.36.3,cargo-hack@0.5.26
Copy link
Member

Choose a reason for hiding this comment

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

Too bad it requires a comma-separated list, an array would be easier to read.

@indietyp indietyp enabled auto-merge March 14, 2023 13:17
@indietyp indietyp added this pull request to the merge queue Mar 14, 2023
Merged via the queue into main with commit d73b75a Mar 14, 2023
@indietyp indietyp deleted the bm/ci/bininstall branch March 14, 2023 14:09
TimDiekmann added a commit that referenced this pull request Mar 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/infra Relates to version control, CI, CD or IaC (area)
Development

Successfully merging this pull request may close these issues.

None yet

2 participants