[ci] Let hook inventory skip data directories - #3572
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## Gga2afs3rc7lmrmvkzm4wdcmi4lwlyi65 #3572 +/- ##
====================================================================
Coverage ? 91.85%
====================================================================
Files ? 20
Lines ? 6093
Branches ? 0
====================================================================
Hits ? 5597
Misses ? 496
Partials ? 0 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
d61725c to
85273d3
Compare
b76c6da to
d6eb90c
Compare
85273d3 to
4f893f6
Compare
|
Authored by an agent, posting via joshlf's account @codex review Please review the current head. Treat any earlier review as stale if it targets |
|
Codex Review: Didn't find any major issues. Keep it up! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
d6eb90c to
da31221
Compare
4f893f6 to
1ae23ca
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Delightful! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
1ae23ca to
499cc9c
Compare
da31221 to
38d4762
Compare
499cc9c to
a392988
Compare
38d4762 to
91a3471
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Delightful! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. 👍 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
32afcb6 to
da05271
Compare
0b6d622 to
ceedb57
Compare
da05271 to
e9449ec
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Hooray! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
e9449ec to
e63e580
Compare
2289e34 to
9b4969a
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Can't wait for the next one! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
e63e580 to
b0ed12c
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. 🚀 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Check only regular files when verifying that pre-push invokes every CI check. Preserve the existing reminder for new scripts while allowing review data to live in a named directory under ci. Tests: ./githooks/pre-push *Authored by an agent, posting via joshlf's account* gherrit-pr-id: Gtynnl3k4vyrtzrvdux7rxs4dkq7l7f6m
b0ed12c to
c07c4e0
Compare
|
Authored by an agent, posting via joshlf's account @codex review Please review the current head, |
|
Codex Review: Didn't find any major issues. More of your lovely PRs please. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
2 participants
Check only regular files when verifying that pre-push invokes every CI
check. Preserve the existing reminder for new scripts while allowing
review data to live in a named directory under ci.
Tests: ./githooks/pre-push
Authored by an agent, posting via joshlf's account
Latest Update: v33 — Compare vs v32
📚 Full Patch History
Links show the diff between the row version and the column version.
⬇️ Download this PR
Branch
git fetch origin refs/heads/Gtynnl3k4vyrtzrvdux7rxs4dkq7l7f6m && git checkout -b pr-Gtynnl3k4vyrtzrvdux7rxs4dkq7l7f6m FETCH_HEADCheckout
git fetch origin refs/heads/Gtynnl3k4vyrtzrvdux7rxs4dkq7l7f6m && git checkout FETCH_HEADCherry Pick
git fetch origin refs/heads/Gtynnl3k4vyrtzrvdux7rxs4dkq7l7f6m && git cherry-pick FETCH_HEADPull
Stacked PRs enabled by GHerrit.