github-action-build-chain icon indicating copy to clipboard operation
github-action-build-chain copied to clipboard

remove node 14

Open shubhbapna opened this issue 2 years ago • 4 comments

Part fix for #405

We don't have to release a new version since we were anyways building it in node 16

MERGE AFTER:

  • https://github.com/kiegroup/build-chain-configuration-reader/pull/93
  • build-chain-configuration-reader is update here

shubhbapna avatar Apr 14 '23 17:04 shubhbapna

but the tool still working with 14.x, why do we need to make it more restrictive? I mean I understand node14 is going EOL but it does not mean the tool is not compatible with 14.x, right?

Ginxo avatar Apr 17 '23 05:04 Ginxo

Right but we will have to start supporting node 18 which might have some breaking changes for node 14 right? (we faced this when we were supporting node 12, 14, and 16 and had to drop 12 because some features like optional chaining in node 16 didn't work with node 12).

For any new feature won't it be enough to make sure that it works in the currently maintained and stable node versions and not the EOL ones?

@Ginxo

shubhbapna avatar Apr 17 '23 19:04 shubhbapna

@Ginxo soon the packages that we depend on will also drop support for node 14, 16:

  • https://github.com/octokit/rest.js/pull/294

And certain packages like gitlab sdk are only supporting node 18 or up now:

  • https://github.com/jdalrymple/gitbeaker/blob/main/packages/rest/package.json#L7

shubhbapna avatar May 15 '23 15:05 shubhbapna

ok, please resolve lock conflicts and we will merge. Thanks mate

Ginxo avatar May 16 '23 08:05 Ginxo