Back

deno #31279

chore: remove version.txt in cli/lib

by dsherret · Feb 06, 2026 at 19:37 UTC · scan-64f64c5a4b513b12

View on Github · Download JSON
Low Risk (0%)

Get this automatically on every PR

Install the Axiomo GitHub App to get Signals as check runs and PR comments on every pull request.

Install App

Risk Assessment

Risk level: Low (0%)

Intent

3/3 criteria met

Remove the version.txt file from cli/lib and update version management.

Acceptance Criteria

  • ✓
    version.txt is removed from cli/lib

    version.txt (-0/+1) file is deleted

  • ✓
    Versioning is correctly updated within cli/lib

    version.rs changed to use CARGO_PKG_VERSION

  • ✓
    Other files reflect updated versioning approach

    Cargo.toml and Cargo.lock updated with new version

Confidence: 90.0% Source: diff analysis AI: openai

Contributors

dsherret PR Author 3 commits + Trusted
Account Age: 5057 days
Prior PRs: 1805
Merged: 1737

Trusted contributor with 1737 merged PRs. maintains 166 public repositories. has 2061 followers. unfamiliar with cli/lib/version.txt.

Evidence

Evidence Completeness: 80.0%
tests_passing Passing
ci_passing Passing
build_successful Passing
lint_passing Passing
Missing: security_scan_clean, coverage_maintained

Supply Chain

Low Risk
Modifies dependencies
Modifies lockfile
Modifies CI config
Modifies build scripts

Focus Files

Review 9 file(s)

tools/deno.lock.json +157

157 lines changed; Configuration

medium
tools/lint.js +22

Source code

medium
cli/lib/version.rs +14

Source code

medium
cli/lib/build.rs +9

Source code

medium
tools/release/01_bump_crate_versions.ts +12

Source code

medium
tools/release/deps.ts +4

Source code

medium
Cargo.toml +2

Configuration

low
cli/lib/Cargo.toml +2

Configuration

low
cli/lib/version.txt +1

Standard file

low

Triage

29

minutes to review

medium

effort level

none

staleness risk

Standard review process

Recommendation

APPROVE 87.0% readiness

Good evidence, acceptable risk, ready for merge

Next Steps

Question

Why is security_scan_clean missing? Consider adding this check.

Question

Why is coverage_maintained missing? Consider adding this check.