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.

Sign up for GitHub

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

Jump to bottom

Remove factory-hooks from default template #599

Closed
dpagini opened this issue Oct 21, 2016 · 1 comment
Closed

Remove factory-hooks from default template #599

dpagini opened this issue Oct 21, 2016 · 1 comment
Assignees
@dpagini
Labels
Enhancement A feature or feature request

Comments

@dpagini
Copy link
Contributor

dpagini commented Oct 21, 2016 •
edited
Loading

Shouldn't the factory-hooks/ folder (and subsequent settings include file) be created as part of the $ blt acsf:init command?
Happy to attempt a PR for this, but wanted to get some feedback before I did anything.
@grasmash
Copy link
Contributor

grasmash commented Nov 7, 2016

Sure, I think that makes sense.

@grasmash grasmash added the Enhancement A feature or feature request label Nov 10, 2016
dpagini added a commit to dpagini/blt that referenced this issue Jan 6, 2017
@dpagini
Fixes acquia#599: remove factory-hooks from default template.
dce3561
@dpagini dpagini mentioned this issue Jan 6, 2017
Merged
dpagini added a commit to dpagini/blt that referenced this issue Jan 6, 2017
@dpagini
Fixes acquia#599: remove factory-hooks from default template.
429b307
grasmash pushed a commit that referenced this issue Jan 6, 2017
@dpagini @grasmash
Fixes #599: remove factory-hooks from default template. (#923)
f645857
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dpagini dpagini

Labels
Enhancement A feature or feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@grasmash @dpagini