Post-release papercuts: fail-closed lockfile check, llvm classifier, version-agnostic docs#353
Open
ciaranra wants to merge 2 commits into
Open
Post-release papercuts: fail-closed lockfile check, llvm classifier, version-agnostic docs#353ciaranra wants to merge 2 commits into
ciaranra wants to merge 2 commits into
Conversation
…S classifier, and make the doctor example version-agnostic
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.
Three post-release papercuts (plan cross-reviewed with Codex; its fail-closed adjustment adopted):
rg --fileslists it despite the ignore, via negation-pattern divergence), and a stale local copy brokejust check-allwith a misleading--lockederror. New predicate: tracked → validate; gitignored (documented generated artifacts, rationale lives in.gitignore) → skip with message; anything else → fail ("commit it or gitignore it with a rationale"), so a lock that should be committed cannot slip through. All three paths tested (including a planted unexpected lock).pecos-rslib-llvmclassifier saidPOSIX :: Linuxbut we publish macOS and Windows wheels — nowOS Independent(matching pecos-rslib; pecos-rslib-cuda deliberately stays Linux-only, cuQuantum is Linux-only).docs/user-guide/cli.mdhardcoded the release version in example output, forcing doc edits on every bump — now av<version>placeholder.