Skip to content

fix: restore release quality gates - #2996

Closed
UnArbosFive wants to merge 1 commit into
release-v441from
fix/release-v441-quality-gates
Closed

fix: restore release quality gates#2996
UnArbosFive wants to merge 1 commit into
release-v441from
fix/release-v441-quality-gates

Conversation

@UnArbosFive

Copy link
Copy Markdown
Contributor

Summary

  • refresh sdk/python/uv.lock for the 11.0.2.dev0 and 0.1.2 package version bumps
  • restore the direct expectation in claim_root_weight with a narrowly scoped lint allowance
  • fix the remaining SDK lock and Rust formatting quality gates on release-v441: bump spec and PyPI bases #2994

Verification

  • uv lock --check
  • cargo fmt --all -- --check
  • SKIP_WASM_BUILD=1 cargo clippy -p node-subtensor-runtime --test claim_root_weight --no-deps -- -D warnings
  • SKIP_WASM_BUILD=1 cargo test -p node-subtensor-runtime --test claim_root_weight

Scope

Stacked on and targeting release-v441 (#2994). #2995 handles the separate Python publication workflow finding.

@vercel

vercel Bot commented Jul 27, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
subtensor Ready Ready Preview, Comment Jul 27, 2026 2:45pm

Request Review

@github-actions

github-actions Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

🛡️ AI Review — Skeptic (security review)

VERDICT: SAFE

VERY HIGH scrutiny (account under 30 days), mitigated by repository admin status and substantial merged history; no Gittensor association found. Branch: fix/release-v441-quality-gates → release-v441.

The diff is small and mechanical: it aligns editable-package lockfile versions with their manifests and changes only test-side error handling with a narrowly scoped Clippy allowance. It does not alter runtime behavior, dependencies, CI execution, or the AI-review trust boundary.

Findings

No findings.

Conclusion

No malicious behavior or security vulnerability was found in the reviewed changes.


🔍 AI Review — Auditor (domain review)

VERDICT: 👍

Gittensor association UNKNOWN; repository admin and established contributor despite a recently created account, with 121 recorded contributions.

The lockfile versions match their source manifests (bittensor 11.0.2.dev0 and bittensor-core 0.1.2), and git diff --check passes. The expect lint allowance is test-only and narrowly scoped. The overlaps with #2994 (this PR’s stacked base) and #2993 (the underlying weight-accounting work) are not duplicate implementations of these quality-gate fixes.

Findings

No findings.

Conclusion

The PR accurately restores the stated quality gates with minimal, consistent changes and is ready to merge.

@github-actions

Copy link
Copy Markdown
Contributor

🔄 AI review updated — Skeptic: SAFE Auditor: 👍

@UnArbosFive

Copy link
Copy Markdown
Contributor Author

Superseded by #2995, which includes the lockfile correction and the remaining release quality-gate fixes in one merge-ready PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant