Skip to content
This repository has been archived by the owner on Oct 20, 2022. It is now read-only.

Use pure ES modules #358

Closed
ehmicky opened this issue Oct 15, 2021 · 1 comment · Fixed by #374
Closed

Use pure ES modules #358

ehmicky opened this issue Oct 15, 2021 · 1 comment · Fixed by #374
Assignees
Labels
type: chore work needed to keep the product and development running smoothly

Comments

@ehmicky
Copy link
Contributor

ehmicky commented Oct 15, 2021

See background at https://github.com/netlify/team-dev/issues/36

Once #357 is done and released, we should use pure ES modules and make a major release.

This is more than just switching from CommonJS to import/export. See this list for other changes which might be involved. This should be broken in many PRs, as much as possible, to lower the risk. Also, non-breaking changes (such as adding file extensions in imports, or loading JSON files differently) should be done before the breaking changes (such as using import/export statements).

At the moment, netlify-redirect-parser is only used by:

@ehmicky ehmicky added the type: chore work needed to keep the product and development running smoothly label Oct 15, 2021
@ehmicky
Copy link
Contributor Author

ehmicky commented Nov 29, 2021

Done in #374

@ehmicky ehmicky self-assigned this Nov 29, 2021
@kodiakhq kodiakhq bot closed this as completed in #374 Dec 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: chore work needed to keep the product and development running smoothly
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant