Skip to content
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

Allow specifying a different hooks directory #1481

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

roiavidan
Copy link

Currently, existing hooks are only sourced from within the root cookiecutter location under the hooks/ folder.
This commit adds a new CLI option to allow overriding that folder, if needed.
The rationale behind it is that one might want to have a generic or common set of pre/post hooks usable by all cookiecutters and sitting outside each cookiecutter template.

@insspb
Copy link
Member

insspb commented Jun 6, 2022

Rebased to master

@insspb insspb added feature This issue/PR relates to major feature request. needs-docs PR Only: This PR require additional documentation labels Jun 6, 2022
@insspb
Copy link
Member

insspb commented Jun 6, 2022

@roiavidan thank you for contribution. Can you add small documentation note on this?

Roi Avidan added 2 commits June 8, 2022 21:13
Currently, existing hooks are only sourced from within the root cookiecutter location under the hooks/ folder.
This commit adds a new CLI option to allow overriding that folder, if needed.
The rationale behind it is that one might want to have a generic or common set of pre/post hooks usable by all cookiecutters and sitting outside each cookiecutter template.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature This issue/PR relates to major feature request. needs-docs PR Only: This PR require additional documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants