Skip to content

Conversation

@wtgodbe
Copy link
Member

@wtgodbe wtgodbe commented Feb 15, 2022

Part of #37889. Resolves the remaining High/Critical 5.0 CompGov bug(s)

@wtgodbe wtgodbe requested review from a team and Pilchie February 15, 2022 00:03
@ghost ghost added this to the 5.0.x milestone Feb 15, 2022
@ghost
Copy link

ghost commented Feb 15, 2022

Hi @wtgodbe. If this is not a tell-mode PR, please make sure to follow the instructions laid out in the servicing process document.
Otherwise, please add tell-mode label.

@wtgodbe wtgodbe added area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework tell-mode Indicates a PR which is being merged during tell-mode labels Feb 15, 2022
@wtgodbe wtgodbe modified the milestones: 5.0.x, 5.0.16 Feb 15, 2022
@Pilchie
Copy link
Member

Pilchie commented Feb 15, 2022

👀

Copy link
Contributor

@dougbu dougbu left a comment

Choose a reason for hiding this comment

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

Looks fine but please don't merge w/o signoff from someone deeper in our node, npm / yarn world.

displayName: Install Node 14.x
inputs:
versionSpec: 10.x
versionSpec: 14.x
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we jump all the way forward to 16.x and catch up w/ 'main'❔ Thoughts @javiercn @BrennanConroy

Copy link
Member Author

Choose a reason for hiding this comment

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

I did this to match 6.0:

- ${{ if eq(parameters.installNodeJs, 'true') }}:
- task: NodeTool@0
displayName: Install Node 14.x
inputs:
versionSpec: 14.x

@wtgodbe wtgodbe merged commit cc2944c into dotnet:release/5.0 Mar 2, 2022
@wtgodbe wtgodbe deleted the wtgodbe/5CompGv branch March 2, 2022 04:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework tell-mode Indicates a PR which is being merged during tell-mode

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants