release: 2.3.0 - #97
Merged
Merged
Conversation
The production-readiness release. The lockstep set moves onto reconverge 0.7.0, which is the half that changes what this product can promise: before it, a cluster- or grid-wide barrier under a guard that is only block-uniform produced no findings at all, so the gate this tool is built on could not see that class. Tests now sit under the part that decides what the number is. The exit-code contract the README states had nothing behind it -- exit 1, "the fastest candidates were refused", is the product's argument in one integer and the one a refactor turns into 0 unnoticed. launchbound-runner had no tests at all, and silently read `--budget-secs 30m` as no budget on the machine that costs money. The release itself had two holes: release.yml never created a GitHub release, so 2.2.1 sat on crates.io for twelve days while the repository showed 2.2.0 as Latest, and the semver baseline was a release behind. Both are gated now rather than remembered. And the supply chain: every action pinned to a SHA, every job capped, credentials not persisted, dependabot watching both ecosystems, and a zizmor job that audits online -- which found a stale pin comment on its first run. Signed-off-by: Vyncint Ng <chivy.nguyen@manabie.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The production-readiness release.
The half that changes what this product can promise
The lockstep set moves onto reconverge 0.7.0 (#70). Before it, a cluster- or grid-wide barrier under a guard that is only block-uniform produced no findings at all — so the gate this tool is built on could not see that class, and every release up to 2.2.1 shipped with an analyzer that could not.
just gateandjust prunewere both re-run at the new set: same verdicts, includingreduce-flip's 3 clean / 8 refused.The measurement path, which was the least tested code here
launchbound-runnerread--budget-secs 30mas no budget — the spellingtune --budgettakes — on the machine the whole stage/ship/run split exists to keep cheap. One parser now, shared with the CLI, which had all of this right alreadymacos-latestnowgpu.ymlthat re-runs the CUDA path — written, not yet exercised, and the issue stays open for thatThe release process, which had let one slip
#74 —
release.ymlhad nogh releasestep at all, so 2.2.1 sat on crates.io for twelve days while the repository showed 2.2.0 as Latest, for the release whose whole content was making the README links work. #75 — the semver baseline was a release behind, against its own written rule;check-versions.shnow holds it to the index.Supply chain and contributor surface
#76–#79: every action a SHA, every job capped from measured durations, credentials not persisted, dependabot on both ecosystems, and a
zizmorjob auditing online — which earned its place on the first run by finding a pin whose# v1comment no longer described it. #80: three issue forms that ask for the provenance a result here depends on, a PR template,CODEOWNERS.Documentation
#73 —
docs/LIMITATIONS.mdcontradicted itself about the T4 and claimed a hardware validation that never happened, in the file the README tells you to read before trusting a result.Gate
cargo test --workspace0 failures, clippy and rustdoc clean at-D warnings,cargo denyok on all four, andcheck-pins,check-versions,links,schemasandskillall green.