Skip to content

bless tests#3214

Closed
joshlf wants to merge 2 commits intomainfrom
G4iagi5qpsy4s5zjljqjzg5p5svspugc3
Closed

bless tests#3214
joshlf wants to merge 2 commits intomainfrom
G4iagi5qpsy4s5zjljqjzg5p5svspugc3

Conversation

@joshlf
Copy link
Copy Markdown
Member

@joshlf joshlf commented Apr 7, 2026


Latest Update: v2 — Compare vs v1

📚 Full Patch History

Links show the diff between the row version and the column version.

Version v1 Base
v2 vs v1 vs Base
v1 vs Base
⬇️ Download this PR

Branch

git fetch origin refs/heads/G4iagi5qpsy4s5zjljqjzg5p5svspugc3 && git checkout -b pr-G4iagi5qpsy4s5zjljqjzg5p5svspugc3 FETCH_HEAD

Checkout

git fetch origin refs/heads/G4iagi5qpsy4s5zjljqjzg5p5svspugc3 && git checkout FETCH_HEAD

Cherry Pick

git fetch origin refs/heads/G4iagi5qpsy4s5zjljqjzg5p5svspugc3 && git cherry-pick FETCH_HEAD

Pull

git pull origin refs/heads/G4iagi5qpsy4s5zjljqjzg5p5svspugc3

Stacked PRs enabled by GHerrit.

This allows developers to isolate their development from their use of
`cargo hermes verify` globally. It also makes development slightly
easier to integrate with Docker, as `cargo run setup`, when run in the
container, doesn't try to write to paths that aren't writeable (this is
specifically a problem since we re-define the UID of the user in the
container in order to match the external UID, making them not root in
the container, but thus making them not have write permissions to root's
home directory).

gherrit-pr-id: Gzjx2m23qp7ycaxvr2hvvsspjslbysoix
Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 Optimistically Approved: Changes appear scoped. Final strict verification will occur in the Merge Queue.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.85%. Comparing base (9252871) to head (db3bbf9).
⚠️ Report is 1 commits behind head on Gzjx2m23qp7ycaxvr2hvvsspjslbysoix.

Additional details and impacted files
@@                        Coverage Diff                         @@
##           Gzjx2m23qp7ycaxvr2hvvsspjslbysoix    #3214   +/-   ##
==================================================================
  Coverage                              91.85%   91.85%           
==================================================================
  Files                                     20       20           
  Lines                                   6067     6067           
==================================================================
  Hits                                    5573     5573           
  Misses                                   494      494           

☔ View full report in Codecov by Sentry.
📢 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.

gherrit-pr-id: G4iagi5qpsy4s5zjljqjzg5p5svspugc3
@joshlf joshlf force-pushed the G4iagi5qpsy4s5zjljqjzg5p5svspugc3 branch from 174688c to db3bbf9 Compare April 7, 2026 04:45
Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 Optimistically Approved: Changes appear scoped. Final strict verification will occur in the Merge Queue.

@joshlf joshlf force-pushed the Gzjx2m23qp7ycaxvr2hvvsspjslbysoix branch 13 times, most recently from da78819 to fc31802 Compare April 7, 2026 23:58
Base automatically changed from Gzjx2m23qp7ycaxvr2hvvsspjslbysoix to main April 8, 2026 01:07
@joshlf joshlf closed this Apr 9, 2026
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