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

Modernisation #47

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Modernisation #47

wants to merge 9 commits into from

Conversation

nickpack
Copy link

Not sure how well used this lib is now, but it fits the bill for some work I am currently doing so I have decided to lightly refactor it so that it works in a more modern stack

  • Minor refactor, targeting ES2021
  • Replaced jshint with eslint (just the default set of rules for now) - no inspections failing
  • All vulnerable dependencies updated and npm audit is now showing no issues.
  • All tests passing
  • Examples tweaked to use import rather than require

I have a bit more work to do to modernise the code fully, but it works in its current state and the test suite is passing.

This will obviously be a pretty significant backwards compatability break so should be considered with caution.

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.

1 participant