Skip to content

Conversation

@JanKrivanek
Copy link
Member

@JanKrivanek JanKrivanek commented Apr 21, 2023

Context

Explict call to ComponentGovernanceComponentDetection@0 task (via /eng/common/templates/steps/component-governance.yml template) based on the branch name (so that this can be invoked for 'vs*' branches as well)

Note

This will be breaking until we consume this arcade update: dotnet/arcade@b1a9b86, it's now flowing to MSBuild main in this PR: #8658

Testing the changes on: https://github.com/dotnet/msbuild/tree/vs-test-only
ADO run: (vs* branch, the CG step injected) https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=7672204&view=logs&j=bb592630-4b9d-53ad-3960-d954a70a95cf&t=424211a8-7b86-5e93-500e-ae39b50be1a7

@JanKrivanek JanKrivanek requested a review from wtgodbe April 21, 2023 18:00
Copy link
Member

@wtgodbe wtgodbe left a comment

Choose a reason for hiding this comment

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

Other than the one change, looks good

@wtgodbe
Copy link
Member

wtgodbe commented Apr 21, 2023

Make sure to port this to all of your vs/* branches as well

@JanKrivanek JanKrivanek requested a review from wtgodbe April 21, 2023 18:08
@JanKrivanek JanKrivanek changed the base branch from main to vs17.2 April 24, 2023 15:47
@JanKrivanek JanKrivanek changed the base branch from vs17.2 to main April 24, 2023 15:48
@JanKrivanek
Copy link
Member Author

Superseded by #8695 (that will eventually flow back to main)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants