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

chore: re-introduce node v14 compatibiliy, bump deps #404

Merged
merged 9 commits into from Dec 27, 2022

Conversation

dnalborczyk
Copy link
Contributor

@dnalborczyk dnalborczyk commented Dec 9, 2022

we have some scenario tests running in the serverless-offline repository with serverless-esbuild. bumping esbuild to v0.15.13+ results into the tests hanging. this was fixed in 0.16.3: evanw/esbuild#2727

unfortunately esbuild-node-externals has a peer dependency on 0.15 as well, which ideally would need to be fixed prior: pradel/esbuild-node-externals#35

we can wait and introduce esbuild v0.16 compat in a follow-up PR.

  • re-introduces node v14 compatibility
  • compiler target to node v14 supported: ES2020
  • bumps all (cjs) deps

@dnalborczyk dnalborczyk marked this pull request as ready for review December 9, 2022 16:57
@dnalborczyk dnalborczyk changed the title ci: re-introduce node v14 compat WIP Dec 9, 2022
@dnalborczyk dnalborczyk marked this pull request as draft December 9, 2022 16:58
@dnalborczyk dnalborczyk changed the title WIP chore: re-introduce node v14 compatibiliy, bump deps Dec 9, 2022
@dnalborczyk dnalborczyk marked this pull request as ready for review December 9, 2022 17:12
@floydspace floydspace merged commit f130bc8 into floydspace:master Dec 27, 2022
@floydspace
Copy link
Owner

@dnalborczyk thank you for your contribution, it makes sense. I made fixes in the e2e tests and ci/cd required to make it work.

@github-actions
Copy link

🎉 This PR is included in version 1.35.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@dnalborczyk
Copy link
Contributor Author

thank you for picking this up @floydspace !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants