Skip to content

Support multiple spec file names in post-checkout hook#11

Merged
albertodebortoli merged 1 commit intomainfrom
auto-update-post-checkout-hook
Apr 8, 2026
Merged

Support multiple spec file names in post-checkout hook#11
albertodebortoli merged 1 commit intomainfrom
auto-update-post-checkout-hook

Conversation

@albertodebortoli
Copy link
Copy Markdown
Member

@albertodebortoli albertodebortoli commented Apr 8, 2026 •
edited
Loading

Summary

  • Extend the post-checkout hook to recognise Lucafile, Lucafile.yml, Toolfile, Toolfile.yml, Skillfile, and Skillfile.yml instead of hardcoding only Lucafile
  • Drop the --spec flag from the luca install call so luca resolves the spec file itself
  • Update the bats test to match the new luca install --quiet invocation

Test Plan

  • All 13 post_checkout.bats tests pass
  • Verify hook exits silently when none of the recognised spec files are present
  • Verify hook triggers synchronization when any of the recognised spec file names are present

@albertodebortoli
Support multiple spec file names in post-checkout hook
4daaae5 
Extend the hook to recognise Lucafile, Lucafile.yml, Toolfile, Toolfile.yml,
Skillfile, and Skillfile.yml instead of only Lucafile. Drop the --spec flag
from the luca install call so luca resolves the spec file itself.
@albertodebortoli albertodebortoli marked this pull request as ready for review April 8, 2026 22:59
@albertodebortoli albertodebortoli merged commit 2be6f85 into main Apr 8, 2026
2 checks passed
@albertodebortoli albertodebortoli deleted the auto-update-post-checkout-hook branch April 8, 2026 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@albertodebortoli