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

Skip merging source files to a single destination file #114

Merged
merged 3 commits into from
Aug 9, 2016

Conversation

aaani
Copy link
Collaborator

@aaani aaani commented Dec 10, 2014

Convert html templates to angular template cache without merging the output together to a single dest file. Output files respect the original folder structure and name of source src files. This behavior is turned off by default and could be turned on by setting merge: true in the options.

@ericclemmons
Copy link
Owner

Dunno if you may be interested in #137...

@underscorebrody
Copy link
Collaborator

@aaani can you please add a test for this scenario? Thanks!

@aaani
Copy link
Collaborator Author

aaani commented Aug 6, 2016

@underscorebrody: Added a test. Thanks.

@underscorebrody
Copy link
Collaborator

@aaani This looks good, just needs a rebase

…ithout merging the output together and thus preserving the source directory structure in the output.
@aaani
Copy link
Collaborator Author

aaani commented Aug 8, 2016

@underscorebrody done!

@underscorebrody
Copy link
Collaborator

This looks like a funky rebase... a bunch of commits here that are not yours. I think because you put a merge commit into it. Can you back out the merge commit? That ought to fix the weirdness.

@aaani
Copy link
Collaborator Author

aaani commented Aug 9, 2016

Sorry about that @underscorebrody. Does it look ok now?

@underscorebrody
Copy link
Collaborator

Yup looks good!

@underscorebrody underscorebrody merged commit bc3980c into ericclemmons:master Aug 9, 2016
@ericclemmons
Copy link
Owner

This is fantastic work! Great job!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants