axodotdev / axodotdev/cargo-dist
[Feature PR] Improve ci configuration for `.github/workflows/release.yml`
- Dominant language
- Rust
- Stars
- 2.1k
- Forks
- 149
- Avg merge
- 1d 11h
- Merged PRs (30d)
- 32
Description
I've found shortcomings in the current `release.yml` file: in the "Install Rust non-interactively if not already installed" section, if `rustc` and `cargo` tools are already installed, the installation step is skipped, ultimately causing the build of the product to fail when the pre-installed tool version is lower than the minimum required by the project.
It caused some errors in my project's Actions tasks:
- https://github.com/Mooling0602/Daemon_Console_Rust/actions/runs/19165853500
- https://github.com/Mooling0602/Daemon_Console_Rust/actions/runs/18746468997
I wrote a better `release.yml` template file, it reads `rust-version` option in `Cargo.toml`, and if current rust (cargo/rustc) version lower than it, then reinstall rust tools to latest version.
I want to make a pull request to merge the `release.yml` file I modified with ChatGPT. I'll do it later and refer to this issue there.
I want to make a contribution to this great project and hope it gets better!
Contributor guide
Research direction
Start with .github/workflows/release.yml, especially the “Install Rust non-interactively if not already installed” section, and inspect Cargo.toml for the rust-version option. Use the linked Actions runs to understand the failure; done means the workflow compares the installed Rust tools with the project minimum and handles an older version before the release build.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github-actions, rust
- Domain
- ci-cd, release
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100