Skip to content

Add SkipRoslynAnalyzers parameter to Start-PSBuild #15640

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.

Sign up for GitHub

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

Jump to bottom
Merged
iSazonov merged 3 commits into from
Jul 7, 2021
Merged

Add SkipRoslynAnalyzers parameter to Start-PSBuild #15640

iSazonov merged 3 commits into from
Jul 7, 2021

Conversation

@xtqqczze
Copy link
Contributor

@xtqqczze xtqqczze commented Jun 22, 2021

Fix #15632

@ghost ghost assigned anmenaga Jun 22, 2021
@iSazonov
Copy link
Collaborator

iSazonov commented Jun 25, 2021

It is not clear will this block source generators?

@iSazonov iSazonov added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Jun 25, 2021
@xtqqczze
Copy link
Contributor Author

xtqqczze commented Jun 25, 2021

It is not clear will this block source generators?

I haven't found any evidence it will.

iSazonov
iSazonov reviewed Jul 1, 2021
View reviewed changes
@iSazonov iSazonov mentioned this pull request Jul 5, 2021
Closed
@xtqqczze xtqqczze changed the title Add SkipAnalyzers parameter to Start-PSBuild Add SkipRoslynAnalyzers parameter to Start-PSBuild Jul 5, 2021
@xtqqczze xtqqczze closed this Jul 6, 2021
@xtqqczze xtqqczze reopened this Jul 6, 2021
@iSazonov iSazonov self-requested a review July 6, 2021 09:42
iSazonov
iSazonov reviewed Jul 6, 2021
View reviewed changes
@iSazonov iSazonov assigned iSazonov and unassigned anmenaga Jul 7, 2021
@iSazonov iSazonov added CL-Tools Indicates that a PR should be marked as a tools change in the Change Log and removed CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log labels Jul 7, 2021
@iSazonov iSazonov merged commit 3a61483 into PowerShell:master Jul 7, 2021
@iSazonov
Copy link
Collaborator

iSazonov commented Jul 7, 2021

@xtqqczze Thanks!

@iSazonov iSazonov added this to the 7.2.0-preview.8 milestone Jul 7, 2021
@xtqqczze xtqqczze deleted the SkipAnalyzers branch July 8, 2021 00:11
@ghost
Copy link

ghost commented Jul 22, 2021

🎉v7.2.0-preview.8 has been released which incorporates this pull request.:tada:

Handy links:

@ghost ghost mentioned this pull request Jul 22, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@iSazonov iSazonov iSazonov approved these changes

@adityapatwardhan adityapatwardhan Awaiting requested review from adityapatwardhan

@anmenaga anmenaga Awaiting requested review from anmenaga

@daxian-dbw daxian-dbw Awaiting requested review from daxian-dbw

@TravisEz13 TravisEz13 Awaiting requested review from TravisEz13

Assignees

@iSazonov iSazonov

Labels

CL-Tools Indicates that a PR should be marked as a tools change in the Change Log

Projects

None yet

Milestone

7.2.0-preview.8

Development

Successfully merging this pull request may close these issues.

Add -SkipAnalyzers switch parameter to Start-PSBuild

3 participants

@xtqqczze @iSazonov @anmenaga