Skip to content

v3.1.1: lint-clean without suppressions, tests build without std, packaging tidied - #13

Merged
GeEom merged 9 commits into
mainfrom
check-gate
Oct 25, 2026
Merged

GeEom merged 9 commits into
mainfrom
check-gate

Conversation

@GeEom

@GeEom GeEom commented Oct 25, 2026 •

Copy link
Copy Markdown
Owner

Patch release. The public API is unchanged, checked by comparing rustdoc JSON item by item against main.

Code (no behaviour change)

  • All #[allow] suppressions are gone. Lints are satisfied with exact equivalents: cast_signed, unsigned_abs, .get() table lookup, and asin_unguarded instead of #[inline(always)].
  • Every public function was checked bit-identical against main over about 30k inputs on 10 type layouts.
  • #[no_panic] now applies only when debug_assertions is off. The no-panic-check profile still proves every public function panic-free.

Docs

  • Long crate and module docs move to docs/*.md and are included with include_str!, so docs.rs shows the same text.
  • Four doc claims that disagreed with the code are fixed:
    • the bounded.rs constructor and AtLeastOne;
    • the circular table's 1/K;
    • tan's overflow distance;
    • the README's exp and pow2 zero thresholds.

Tests and CI

  • Tests return Result instead of unwrapping, using the crate's own Error, so cargo test --no-default-features builds. The publish workflow requires that, and CI now runs it too.
  • Each CI job sets RUSTUP_TOOLCHAIN, so the committed rust-toolchain.toml (1.99.1, for local checks) doesn't override the matrix, MSRV or publish toolchains.
  • Cargo.lock is committed. crossbeam-epoch (dev-only) moves off a yanked release.

Packaging and release

  • The derivation notes, rust-toolchain.toml, clippy.toml, codecov.yml and .github/ are excluded from the crate.
  • The release step uses softprops/action-gh-release@v2.

Checked locally

  • cargo test with default and no default features;
  • RUSTUP_TOOLCHAIN=1.95 cargo check --locked --features std;
  • cargo package;
  • the no-panic link proof.

GeEom added 9 commits October 25, 2026 17:40
… and clippy config, and deny the lints it requires
…y bound; long crate and module docs move to docs/ and are included, so docs.rs shows the same text
…suppressions, and keep only comments that explain why
…t the std feature, as the publish workflow's cargo test --no-default-features requires.
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 96.15385% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/kernel/cordic.rs 75.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@GeEom
GeEom merged commit 5743307 into main Oct 25, 2026
12 checks passed
@GeEom
GeEom deleted the check-gate branch October 25, 2026 22:24
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.

2 participants