-
Notifications
You must be signed in to change notification settings - Fork 10
refactor(stack): extract _get_hooks_dir() helper #934
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor(stack): extract _get_hooks_dir() helper #934
Conversation
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🟢 🤖 Continuous IntegrationWonderful, this rule succeeded.
🟢 👀 Review RequirementsWonderful, this rule succeeded.
🟢 Enforce conventional commitWonderful, this rule succeeded.Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
🟢 🔎 ReviewsWonderful, this rule succeeded.
🟢 📕 PR descriptionWonderful, this rule succeeded.
|
🧪 CI InsightsHere's what we observed from your CI run for 2c041d4. 🟢 All jobs passed!But CI Insights is watching 👀 |
d2b3175 to
843ec91
Compare
45260a8 to
41bf5c4
Compare
843ec91 to
dd362a1
Compare
41bf5c4 to
e16a5ee
Compare
dd362a1 to
c3dbb6e
Compare
e16a5ee to
50c44f7
Compare
Pull request has been modified.
c3dbb6e to
22edf5f
Compare
50c44f7 to
cafc24a
Compare
|
@jd this pull request is now in conflict 😩 |
Consolidate the duplicated hooks directory resolution logic into a single async helper function. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> Change-Id: I0b88b1f5a7a332665b51fbeda5341c80b2ab40e4 Claude-Session-Id: 3dabd187-059b-40b7-9a52-a2542a27c752
cafc24a to
2c041d4
Compare
Merge Queue Status✅ The pull request has been merged at 2c041d4 This pull request spent 1 minute 57 seconds in the queue, including 1 minute 43 seconds running CI. Required conditions to merge
|
Consolidate the duplicated hooks directory resolution logic into a
single async helper function.
Co-Authored-By: Claude Opus 4.5 noreply@anthropic.com