Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Infrastructure] Add a workflow to validate package-lock.json when changes are introduced. #54688

Merged
merged 9 commits into from Mar 22, 2024

Conversation

javiercn
Copy link
Member

This forces a check against the azdo feed at the time the changes are introduced

@dotnet-issue-labeler dotnet-issue-labeler bot added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Mar 22, 2024
Copy link
Member Author

@javiercn javiercn left a comment

Choose a reason for hiding this comment

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

Perform a bare checkout and avoid checking out submodules

@javiercn javiercn closed this Mar 22, 2024
@javiercn javiercn reopened this Mar 22, 2024
@dotnet-policy-service dotnet-policy-service bot added this to the 9.0-preview4 milestone Mar 22, 2024
@javiercn javiercn marked this pull request as ready for review March 22, 2024 12:31
@javiercn javiercn requested review from wtgodbe and a team as code owners March 22, 2024 12:31
@javiercn javiercn enabled auto-merge (squash) March 22, 2024 13:04
Copy link
Member

@SteveSandersonMS SteveSandersonMS left a comment

Choose a reason for hiding this comment

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

Looks great!

I'm not clear on whether this PR actually validates that the new workflow succeeds, since it only runs the new workflow when package-lock.json is modified, and it's not modified in this PR. Assuming you know the new workflow will succeed in general, LGTM!

@javiercn javiercn merged commit 35e9982 into main Mar 22, 2024
22 checks passed
@javiercn javiercn deleted the javiercn/validate-package-lock-json-against-upstream branch March 22, 2024 16:20
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants