release.yml skips cargo-semver-checks when crates.io answers with any error

Open Beginner friendly
#130 0 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
78/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Active
Tech stack
github-actions, rust
Domain
ci-cd, release

Research direction

Read the “Look for a published baseline” step around lines 78–86 of release.yml and inspect the relevant checks in ci-scripts. Make the step distinguish a 404 from other non-200 responses, with non-404 errors failing or retrying. Done when a stand-in returning 503 fails the step.

Written by the indexing model from the issue text.

Description

bug github_actions

Today — the "Look for a published baseline" step (release.yml ~78-86) sets found=true only for HTTP 200, so a 429, 500 or 503 from crates.io is read as "first release, no baseline" and the semver gate is skipped before publishing.

Fix — distinguish 404 (no crate) from any other non-200 (fail the job, or retry with backoff); since every crate now exists, a missing baseline should fail rather than skip.

Done when — a stand-in returning 503 fails the step; tested in ci-scripts.

Found in the 0.3.0 review; published 0.2.0 has the same code.

Dominant language
Rust
Stars
1
Forks
2
Avg merge
5d 22h
Merged PRs (30d)
4

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from vyncint/oxidelake

All issues in vyncint/oxidelake

Similar issues

More Rust issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.