oxidecomputer / oxidecomputer/omicron

ensure clippy (helios) and docs jobs are land-blocking

Open
#7,069 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Rust
Stars
572
Forks
97
Avg merge
2d 12h
Merged PRs (30d)
96

Description

#7022 broke these jobs and landed on main (sorry!) -- we should mark them required.

(And as iliana points out in chat, we should try and mark all jobs required.)

Contributor guide

Open the contributing guide

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.

Research direction

Inspect the repository's CI workflow definitions and identify the clippy (helios) and docs jobs mentioned in the issue. Start by comparing their status-check configuration with the behavior introduced by #7022. Done means both jobs are required for landing, with the broader question of making all jobs required addressed or explicitly scoped.

Written by the indexing model from the issue text.

Assessment

Tech stack
github-actions, rust
Domain
ci-cd
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.