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

Upgrade major deps, migrate package to ESM #5

Merged
merged 2 commits into from
Jan 21, 2022

Conversation

crutchcorn
Copy link
Contributor

This PR upgrades all major deps, which has resulted in migrating the package to ESM

A good article that we could link to as a changelog is here: https://gist.github.com/sindresorhus/a39789f98801d908bbc7ff3ecc99d99c

Tests pass and this should be a fully functional upgrade. I have a TS typings PR to make immediately following this one, but will wait until this one is merged to be able to add another onto the fray :)

@ksoichiro ksoichiro merged commit 168b2e8 into ksoichiro:master Jan 21, 2022
@ksoichiro
Copy link
Owner

Thank you! Sorry for my late response 🙇
Migrating to ESM sounds good, but I'd like to support CommonJS as well for the time being because it seems nuxt/content doesn't support ESM plugins yet. I'll handle it later.

@ksoichiro
Copy link
Owner

Handled in #7

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.

None yet

2 participants