Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

SA1002 The spacing around a semicolon is incorrect - #7199

#7199
Merged
Forgind merged 4 commits into
dotnet:maindotnet/msbuild:mainfrom
elachlan:SA1002elachlan/msbuild:SA1002Copy head branch name to clipboard
Jan 11, 2022
Merged

SA1002 The spacing around a semicolon is incorrect#7199
Forgind merged 4 commits into
dotnet:maindotnet/msbuild:mainfrom
elachlan:SA1002elachlan/msbuild:SA1002Copy head branch name to clipboard

Conversation

@elachlan

@elachlan elachlan commented Jan 2, 2022

Copy link
Copy Markdown
Contributor

@sharwell sharwell left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

💡 Consider increasing the severity of SA1002 to warning as part of this pull request

@Forgind Forgind added the merge-when-branch-open PRs that are approved, except that there is a problem that means we are not merging stuff right now. label Jan 10, 2022
@Forgind
Forgind merged commit 4e10077 into dotnet:main Jan 11, 2022
@elachlan
elachlan deleted the SA1002 branch January 11, 2022 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-when-branch-open PRs that are approved, except that there is a problem that means we are not merging stuff right now.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.