Skip to content

chore(CI): bump node version#1116

Merged
czosel merged 1 commit intomainfrom
bump-node
Jun 13, 2023
Merged

chore(CI): bump node version#1116
czosel merged 1 commit intomainfrom
bump-node

Conversation

@czosel
Copy link
Copy Markdown
Collaborator

@czosel czosel commented Jun 7, 2023

Node 14 is EOL. @cseufert strictly speaking this is a breaking change - should we treat it as such?

@cseufert
Copy link
Copy Markdown
Collaborator

cseufert commented Jun 7, 2023

We are not yet breaking on node14 are we? Do we need to hard drop support for it, or should we wait until a dependency/feature can no longer support node 14?

@czosel
Copy link
Copy Markdown
Collaborator Author

czosel commented Jun 8, 2023

Totally, we aren’t. Shall we leave node 14 in CI until it breaks then?

@cseufert
Copy link
Copy Markdown
Collaborator

cseufert commented Jun 9, 2023

Yes, I think we leave it, and once it breaks our next release will drop support for it.

Comment thread .github/workflows/test.yml Outdated
Copy link
Copy Markdown
Collaborator

@cseufert cseufert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@czosel czosel merged commit 66c2cc8 into main Jun 13, 2023
@czosel czosel deleted the bump-node branch June 13, 2023 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants