Movatterモバイル変換


[0]ホーム

URL:


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

Sign up
Appearance settings

Update AnalyzerReleases file to reflect new analyzers for .NET8 release#6940

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
mavasani merged 1 commit intodotnet:mainfrommavasani:UpdateAnalyzerReleases
Sep 15, 2023

Conversation

@mavasani
Copy link

Note that this PR is essential to ensure setting AnalysisLevel to 8 enables the new analyzers added with .NET8.

Note that this PR is essential to ensure setting AnalysisLevel to 8 enables the new analyzers added with .NET8.
@mavasanimavasani requested a review froma team as acode ownerSeptember 14, 2023 06:29
@mavasani
Copy link
Author

@carlossanlop This is another PR that would need to be backported to release 8.0.1.xx branch.

@codecov
Copy link

codecovbot commentedSep 14, 2023

Codecov Report

Merging#6940 (77351a2) intomain (d29b797) willincrease coverage by0.00%.
Report is 2 commits behind head on main.
The diff coverage isn/a.

@@           Coverage Diff           @@##             main    #6940   +/-   ##=======================================  Coverage   96.40%   96.40%           =======================================  Files        1402     1402             Lines      331927   331932    +5       Branches    11031    11032    +1     =======================================+ Hits       319979   319991   +12+ Misses       9186     9178    -8- Partials     2762     2763    +1

@mavasanimavasani merged commitb0e8ed7 intodotnet:mainSep 15, 2023
@mavasani
Copy link
Author

/backport to release/8.0.1xx

@github-actions
Copy link

Started backporting to release/8.0.1xx:https://github.com/dotnet/roslyn-analyzers/actions/runs/6193037384

@github-actions
Copy link

@mavasani backporting to release/8.0.1xx failed, the patch most likely resulted in conflicts:

$ git am --3way --ignore-whitespace --keep-non-patch changes.patchApplying: Update AnalyzerReleases file to reflect new analyzersfor .NET8 releaseUsing index info to reconstruct a base tree...Msrc/NetAnalyzers/Core/AnalyzerReleases.Unshipped.mdFalling back to patching base and 3-way merge...Auto-merging src/NetAnalyzers/Core/AnalyzerReleases.Unshipped.mdCONFLICT (content): Merge conflictin src/NetAnalyzers/Core/AnalyzerReleases.Unshipped.mderror: Failed to mergein the changes.hint: Use'git am --show-current-patch=diff' to see the failed patchPatch failed at 0001 Update AnalyzerReleases file to reflect new analyzersfor .NET8 releaseWhen you have resolved this problem, run"git am --continue".If you prefer to skip this patch, run"git am --skip" instead.To restore the original branch and stop patching, run"git am --abort".Error: The process'/usr/bin/git' failed withexit code 128

Please backport manually!

@github-actions
Copy link

@mavasani an error occurred while backporting to release/8.0.1xx, please check the run log for details!

Error: git am failed, most likely due to a merge conflict.

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

Reviewers

1 more reviewer

@carlossanlopcarlossanlopcarlossanlop approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mavasani@carlossanlop@davepcallan

[8]ページ先頭

©2009-2025 Movatter.jp