[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% 91.85%
==================================================================
Files 20 20
Lines 6093 6093
==================================================================
Hits 5597 5597
Misses 496 496 ☔ 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". |
*Authored by Codex, posting via joshlf's account* 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. gherrit-pr-id: Gtynnl3k4vyrtzrvdux7rxs4dkq7l7f6m Tests: ./githooks/pre-push.
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". |
2 participants
Authored by Codex, posting via joshlf's account
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.
Latest Update: v4 — Compare vs v3
📚 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.