Skip to content

Conversation

@adrinr
Copy link
Member

@adrinr adrinr commented Dec 2, 2024

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:

  1. Fork, use and migrate all the projects
  2. Copy the code that is actually required into this project

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.

@adrinr adrinr requested review from mike12345567 and samwho December 2, 2024 16:31
Copy link

@samwho samwho left a 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:
Copy link

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?

Copy link
Member Author

@adrinr adrinr Dec 3, 2024

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

@adrinr
Copy link
Member Author

adrinr commented Dec 3, 2024

For historical purposes could you detail the rationale for this change in the PR description? 🙏

Good shout! Done

@adrinr adrinr requested a review from samwho December 3, 2024 13:14
@adrinr adrinr merged commit f3735de into master Dec 4, 2024
8 checks passed
@adrinr adrinr deleted the remove-handlebars-utils branch December 4, 2024 12:48
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