Raise the minimum required version of node (and transition to ESM)

[gr2m]

See for reference: semantic-release/semantic-release#2055

• require a minimum node version of the latest LTS, currently 14.17
• ensure engines-strict is enforced in the CI of each project to reduce the cases where incompatible dependencies are installed outside of our supported range (example)
• Make sure to update the docs if we mention a minimal node version anywhere

Bonus:

• Make it a native ES Module, but wait until we decided to go with a native ESM module as part of Raise the minimum required version of node in semantic-release/semantic-release semantic-release#2055

Pull request welcome! Just comment below if you'd like to work on it and we will assign the issue to you. Thank you for your help 💐

[abel-mak]

Hi, is modification of the minimum required version should be in package.json in engines?

[gr2m]

yes, but there are more places you need to change. See this pull request for comparison, it also includes the change to ESM:
semantic-release/commit-analyzer#248

Here is the commit which bumps the minimal required version:
semantic-release/commit-analyzer@e7e56f6

[abel-mak]

the changes should be in beta branch?

[gr2m]

no, but the pull request should be opened against the beta branch

[abel-mak]

Hey, i tried with esm part but some test are not passing, related to proxyquire

[gr2m]

Ah yes we've seen this elsewhere. @travi suggested an alternative. Can you start a pull request with your current changes and we take it from there?

[gr2m]

found the comment. Can you try to replace proxyquire with https://github.com/testdouble/testdouble.js?

[abel-mak]

the problem is this line

const {generateNotes} = proxyquire('..', {'conventional-changelog-writer': writer});

should it be?

const {generateNotes} = await td.replaceEsm('../index.js');

[gr2m]

should it be?

const {generateNotes} = await td.replaceEsm('../index.js');

I don't know, I havent used testdouble myself yet. Could you give it a try and see if the tests pass?

[abel-mak]

not it doesn't, but now the error is not related to that line, but instead this

  t.deepEqual(writer.args[0][0], {
    version: nextRelease.version,
    host,
    owner,
    repository,
    previousTag: lastRelease.gitTag,
    currentTag: nextRelease.gitTag,
    linkCompare: lastRelease.gitTag,
    issue: 'issues',
    commit: 'commit',
    packageData: undefined,
    linkReferences: undefined,
  });

Error:

 TypeError {
    message: 'Cannot read property \'0\' of undefined',
  }

[gr2m]

I can't look into it myself before Aug 3rd. I'd appreciate if you or someone else could dig into it and try to figure out the errors

[github-actions]

🎉 This issue has been resolved in version 10.0.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀