Skip to content
This repository has been archived by the owner on Oct 19, 2023. It is now read-only.

Auto-require and register partials (+tests) #47

Merged
merged 3 commits into from
Sep 9, 2015

Conversation

kirbysayshi
Copy link
Collaborator

Hello! This PR adds support for implicit require of Handlebars partials. This way you can do something like:

{{> some-package/a-partial.hbs}}

And it will automatically be requireed by browserify and then registered as a partial.

Since this is a large-ish change, I completely understand if you don't want to merge this / maintain this functionality. If so, just let me know! I'll probably do some sort of hard fork (with a completely different name, of course). If you are into it, let me know and I'll add some documentation :) And of course, please let me know your general thoughts as well.

Thanks!

@kirbysayshi
Copy link
Collaborator Author

Just noticed the build failed. Looks like the travis config is out of date since it's using a very old npm and node.

@kirbysayshi
Copy link
Collaborator Author

@epeli Heyo, just checking in. Do you think you'll have time to take a look at this PR any time soon?

kirbysayshi added a commit that referenced this pull request Sep 9, 2015
Auto-require and register partials (+tests)
@kirbysayshi kirbysayshi merged commit 5367cd7 into esamattis:master Sep 9, 2015
@kirbysayshi kirbysayshi deleted the ksh/require-traverse branch September 9, 2015 21:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant