Migrate to use of ES Modules

getgauge / gauge-js

JavaScript language plugin for Gauge

MIT License

81 stars 39 forks source link

Migrate to use of ES Modules #667

Closed chadlwilson closed 2 months ago

chadlwilson commented 2 months ago

Moves minimum node version to v18 (latest LTS at time of PR)
Changes gauge-js to an ES module, removing all use of CommonJS requires
Prefers use of node: prefixed imports
Upgrades and de-duplicates dependencies to latest

chadlwilson commented 2 months ago

Right now this logic is a problem: https://github.com/getgauge/gauge-js/blob/81ad5c1cea711949bac7b487e33ff4163e86eef3/src/req-manager.js#L18-L50

chadlwilson commented 2 months ago

Thanks for the additional work here!

Apologies, I rebased and force-pushed without thinking (after pushing removal of dead connection.js code to master separately since it's not coupled), so you probably need to hard reset from origin here if there is more work to be done. Apologies.

gaugebot[bot] commented 2 months ago

@chadlwilson Thank you for contributing to gauge-js. Your pull request has been labeled as a release candidate 🎉🎉.

Merging this PR will trigger a release.

Please bump up the version as part of this PR.

Instructions to bump the version can found at CONTRIBUTING.md

If the CONTRIBUTING.md file does not exist or does not include instructions about bumping up the version, please looks previous commits in git history to see what changes need to be done.