-
Notifications
You must be signed in to change notification settings - Fork 17
Remove handlebars-utils package #23
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For historical purposes could you detail the rationale for this change in the PR description? 🙏
| glogg "^1.0.0" | ||
|
|
||
| handlebars-utils@^1.0.4, handlebars-utils@^1.0.6: | ||
| handlebars-utils@^1.0.4: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this not entirely be gone now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Due the maze of dependencies, this is still used for a dev package called "verb". This should probably not be needed, but given that is a dev dependency we should be good for now
Good shout! Done |
Remove handlebars-utils package. The newly introduced code comes from the removed package, being copied in here.
The original repo is making use of a lot of small packages that depend on each other. As we want to migrate to ES, we have 2 options:
Given that the packages are quite small and most of them don't do much, we decided to copy the code over. This will also allow us refactoring and cleaning the code much easier.