Skip to content

Prepare uphold 1.23.0 - #289

Merged
HackingGate merged 1 commit into
mainfrom
prepare-1.23.0
Sep 30, 2026
Merged

HackingGate merged 1 commit into
mainfrom
prepare-1.23.0

Conversation

@HackingGate

Copy link
Copy Markdown
Owner

One engine change since 1.22.0. A pattern rule may declare files.reach = "pinned": the scan then reads the gitlinks from the repository's index and
runs git ls-files inside each pinned member, so a superproject's rule sees
the content it pins, reported under the mount path. Absent, or
"repository", is 1.22.0's behavior byte for byte. A pinned mount whose
working tree is absent, never initialised or marked inactive, is exit 2
naming the mount and git submodule update --init <path>; pins are followed
at every depth; each member answers for its own .gitattributes; findings
and baselines key on the mount-prefixed path; and include, exclude and glob
keep gitignore semantics rooted at the superproject. A member never borrows
its superproject's policy, as before (ADR 0012, Accepted).

The repository's own decisions now go to a Design record issue, and
docs/adr is frozen by a path rule in its own policy.

A consumer taking the pin to v1.23.0 needs no change. A consumer that opts a
rule into "pinned" must check out every pinned member wherever that rule
runs, CI included, and pays about 9 ms per member once per run.

After merge the owner tags: git tag v1.23.0 && git push --tags.

https://claude.ai/code/session_01DzvkN2qyaQDc3h7vqY2zLL

One engine change since 1.22.0. A pattern rule may declare `files.reach =
"pinned"`: the scan then reads the gitlinks from the repository's index and
runs `git ls-files` inside each pinned member, so a superproject's rule sees
the content it pins, reported under the mount path. Absent, or
`"repository"`, is 1.22.0's behavior byte for byte. A pinned mount whose
working tree is absent, never initialised or marked inactive, is exit 2
naming the mount and `git submodule update --init <path>`; pins are followed
at every depth; each member answers for its own `.gitattributes`; findings
and baselines key on the mount-prefixed path; and include, exclude and glob
keep gitignore semantics rooted at the superproject. A member never borrows
its superproject's policy, as before (ADR 0012, Accepted).

The repository's own decisions now go to a Design record issue, and
`docs/adr` is frozen by a path rule in its own policy.

A consumer taking the pin to v1.23.0 needs no change. A consumer that opts a
rule into `"pinned"` must check out every pinned member wherever that rule
runs, CI included, and pays about 9 ms per member once per run.

Claude-Session: https://claude.ai/code/session_01DzvkN2qyaQDc3h7vqY2zLL
@coderabbitai

coderabbitai Bot commented Sep 30, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 33 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 7b0a8ca2-70ae-4bf4-9f5d-c85abd8e61b6

📥 Commits

Reviewing files that changed from the base of the PR and between 80b2e47 and afe87f6.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (3)
  • Cargo.toml
  • README.md
  • hooks/lefthook.yml
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.15%. Comparing base (80b2e47) to head (afe87f6).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #289   +/-   ##
=======================================
  Coverage   94.15%   94.15%           
=======================================
  Files          46       46           
  Lines       20547    20547           
=======================================
  Hits        19345    19345           
  Misses       1202     1202           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@HackingGate
HackingGate merged commit 1036212 into main Sep 30, 2026
12 checks passed
@HackingGate
HackingGate deleted the prepare-1.23.0 branch September 30, 2026 15:39
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