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

refactor(lib): update dependencies, replace deprecated #6

Merged
merged 22 commits into from
Jul 13, 2022

Conversation

mihai-ro
Copy link
Contributor

@mihai-ro mihai-ro commented May 18, 2022

Work done

  • refactor(dependencies): replace deprecated dependencies. bump versions to latest. replace tslint with eslint. add lint script.
  • ci(babelrc): target node@16. resolve corejs deps on entry. reference plugin-transform-runtime
  • style(static code analyser): define eslint and eslint ignore. cleanup prettierrc
  • style(misc): fix lint issues in files
  • chore(dist): update dist folder with latest build
  • chore(dist): update dist with builtins on usage

@mihai-ro mihai-ro changed the title refactor(lib): refactor(lib): update dependencies, replace deprecated May 18, 2022
@csandman
Copy link
Owner

Thanks for this, I have let this package's maintenance fall behind a bit and you have made a lot of updates that I would have liked to myself. I will review this when I get a chance and (after tweaking a few things) merge it.

@csandman csandman closed this May 31, 2022
@csandman csandman reopened this May 31, 2022
@csandman
Copy link
Owner

Accidental close, reopened

@mihai-ro
Copy link
Contributor Author

mihai-ro commented Jul 2, 2022

@csandman when can we merge this?

@csandman csandman merged commit 3a5b2b3 into csandman:main Jul 13, 2022
@csandman
Copy link
Owner

@mihai-ro After a couple tweaks I merged it. On a side note, if you want PRs to be merged faster, keeping them smaller and more focused on a specific change makes that more likely.

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.

2 participants