[engsys] update platform matrix for node 14 deprecation #27302
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
xirzec
requested review from
ckairen,
mikeharder,
weshaggard and
benbp
as code owners
|
A few other matrices where 14.x is used: |
xirzec
requested review from
timovv,
miguhern,
whisper6284,
RoyHerrod and
danielav7
as code owners
|
API change check API changes are not detected in this pull request. |
benbp
approved these changes
Oct 2, 2023
jeremymeng
approved these changes
Oct 2, 2023
|
The keyvault failure looks unrelated since it's a recorder issue in secrets and I didn't change the platform configuration for it. I think I'm going to go ahead and merge to get this in, but /cc @timovv for awareness, so he might disparage me for my sins if I am regressing anything |
|
/check-enforcer override |
jeremymeng
pushed a commit
to jeremymeng/azure-sdk-for-js
that referenced
this pull request
Nov 7, 2023
[engsys] update platform matrix for node 14 deprecation (Azure#27302) Update the CI matrix so we are no longer testing on Node 14 as it has reached the end of our EOL + 6 months policy. For coverage and browser legs I bumped it to use the current LTS (18) rather than 16 which is EOL, since most customers are not expected to be using an out of support runtime.
jeremymeng
added a commit
that referenced
this pull request
Nov 7, 2023
[engsys] update platform matrix for node 14 deprecation (#27302) Update the CI matrix so we are no longer testing on Node 14 as it has reached the end of our EOL + 6 months policy. For coverage and browser legs I bumped it to use the current LTS (18) rather than 16 which is EOL, since most customers are not expected to be using an out of support runtime. Co-authored-by: Jeff Fisher <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Describe the problem that is addressed by this PR
Update the CI matrix so we are no longer testing on Node 14 as it has reached the end of our EOL + 6 months policy.
For coverage and browser legs I bumped it to use the current LTS (18) rather than 16 which is EOL, since most customers are not expected to be using an out of support runtime.