Skip to content

Introducing new flag --return-success-on-findings. - #1756

Merged
josefpihrt merged 5 commits into
dotnet:mainfrom
mdrybak:feature/1693_return_0_on_findings
Aug 1, 2026
Merged

Introducing new flag --return-success-on-findings.#1756
josefpihrt merged 5 commits into
dotnet:mainfrom
mdrybak:feature/1693_return_0_on_findings

Conversation

@mdrybak

@mdrybak mdrybak commented Mar 29, 2026

Copy link
Copy Markdown
Contributor

Fixes #1693.
New flag: --return-success-on-findings is introduced. When it's used, CLI will return 0 even if findings are reported.
Naming isn't really creative and I'm not really emotionally attached to it, so feel free to suggest some new name and I'll just change it.
Tagging @davewichers as the issue reporter.

mdrybak added 3 commits March 29, 2026 20:56
@mdrybak
Addressing issue dotnet#1693, introducing new flag --return-success-o…
ef478d2 
…n-findings. With this flag, CLI will return exit code 0 even when findings are reported.
@mdrybak
Entry in changelog.
36463e3
@mdrybak
PR number adjustment.
71137b0
josefpihrt
josefpihrt requested changes Aug 1, 2026
View reviewed changes
Comment thread ChangeLog.md Outdated
Comment thread src/CommandLine/Options/AnalyzeCommandLineOptions.cs Outdated
Comment thread src/CommandLine/Commands/AnalyzeCommand.cs Outdated
Comment thread src/CommandLine/Options/AnalyzeCommandLineOptions.cs Outdated
@josefpihrt
josefpihrt merged commit 646d15c into dotnet:main Aug 1, 2026
17 checks passed
@josefpihrt

josefpihrt commented Aug 1, 2026

Copy link
Copy Markdown
Collaborator

@mdrybak Thank you for your contribution!

I changed the name to --return-success-on-diagnostics.

@davewichers

davewichers commented Aug 3, 2026

Copy link
Copy Markdown

@mdrybak - Thanks for addressing this. @josefpihrt - When do you expect this new feature to be released in a new version of Roslynator?

@josefpihrt

josefpihrt commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

@davewichers this week for sure.

This was referenced Aug 8, 2026
Closed
Closed
Open
Merged
Merged
Open
Closed
Merged
Merged
Open
Open
Open
Open
Open
Open
Merged
Merged
Open
Open
This was referenced Aug 10, 2026
Open
Open
Open
vdolek pushed a commit to EvilBrainsTechnology/EvilCase that referenced this pull request Aug 10, 2026
@dependabot
Bump the analyzers group with 3 updates (#272)
5b13881 
Updated [Roslynator.Analyzers](https://github.com/dotnet/roslynator)
from 4.15.0 to 4.16.0.

<details>
<summary>Release notes</summary>

_Sourced from [Roslynator.Analyzers's
releases](https://github.com/dotnet/roslynator/releases)._

## 4.16.0

### Added

- [CLI] Suppress error code from Roslynator when it detects issues in
code but runs successfully
([PR](dotnet/roslynator#1756) by @​mdrybak)

### Fixed

- Fix analyzer
[RCS1118](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1118)
to not report local variable passed as 'in' argument
([PR](dotnet/roslynator#1782) by @​NoahStolk)
- Fix analyzer
[RCS1074](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1074)
([PR](dotnet/roslynator#1768) by @​cbersch)
- Fix enum contained flags check for partial matches in
[RCS1258](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1258)
([PR](dotnet/roslynator#1740) by @​ovska)
- Fix analyzer
[RCS1146](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1146)
([PR](dotnet/roslynator#1747))
- Fix analyzer
[RCS1194](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1194)
([PR](dotnet/roslynator#1733))
- Fix analyzer
[RCS1060](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1060)
to ignore classes marked with `file` modifier
([PR](dotnet/roslynator#1777) by @​cbersch)
- Fix analyzer
[RCS1231](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1231)
([PR](dotnet/roslynator#1774) by @​cbersch)
- Fix analyzer
[RCS1246](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1246)
for conditional access expressions
([PR](dotnet/roslynator#1772 by @​krajek))
- [CLI] Fix `fix` command ignoring `--include` / `--exclude` file filter
([PR](dotnet/roslynator#1758) by @​hashiiiii)
- [CLI] Fix loading of projects and solutions on .NET 10 SDK
([PR](dotnet/roslynator#1783))


Commits viewable in [compare
view](dotnet/roslynator@v4.15.0...v4.16.0).
</details>

Updated
[Roslynator.CodeAnalysis.Analyzers](https://github.com/dotnet/roslynator)
from 4.15.0 to 4.16.0.

<details>
<summary>Release notes</summary>

_Sourced from [Roslynator.CodeAnalysis.Analyzers's
releases](https://github.com/dotnet/roslynator/releases)._

## 4.16.0

### Added

- [CLI] Suppress error code from Roslynator when it detects issues in
code but runs successfully
([PR](dotnet/roslynator#1756) by @​mdrybak)

### Fixed

- Fix analyzer
[RCS1118](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1118)
to not report local variable passed as 'in' argument
([PR](dotnet/roslynator#1782) by @​NoahStolk)
- Fix analyzer
[RCS1074](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1074)
([PR](dotnet/roslynator#1768) by @​cbersch)
- Fix enum contained flags check for partial matches in
[RCS1258](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1258)
([PR](dotnet/roslynator#1740) by @​ovska)
- Fix analyzer
[RCS1146](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1146)
([PR](dotnet/roslynator#1747))
- Fix analyzer
[RCS1194](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1194)
([PR](dotnet/roslynator#1733))
- Fix analyzer
[RCS1060](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1060)
to ignore classes marked with `file` modifier
([PR](dotnet/roslynator#1777) by @​cbersch)
- Fix analyzer
[RCS1231](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1231)
([PR](dotnet/roslynator#1774) by @​cbersch)
- Fix analyzer
[RCS1246](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1246)
for conditional access expressions
([PR](dotnet/roslynator#1772 by @​krajek))
- [CLI] Fix `fix` command ignoring `--include` / `--exclude` file filter
([PR](dotnet/roslynator#1758) by @​hashiiiii)
- [CLI] Fix loading of projects and solutions on .NET 10 SDK
([PR](dotnet/roslynator#1783))


Commits viewable in [compare
view](dotnet/roslynator@v4.15.0...v4.16.0).
</details>

Updated
[Roslynator.Formatting.Analyzers](https://github.com/dotnet/roslynator)
from 4.15.0 to 4.16.0.

<details>
<summary>Release notes</summary>

_Sourced from [Roslynator.Formatting.Analyzers's
releases](https://github.com/dotnet/roslynator/releases)._

## 4.16.0

### Added

- [CLI] Suppress error code from Roslynator when it detects issues in
code but runs successfully
([PR](dotnet/roslynator#1756) by @​mdrybak)

### Fixed

- Fix analyzer
[RCS1118](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1118)
to not report local variable passed as 'in' argument
([PR](dotnet/roslynator#1782) by @​NoahStolk)
- Fix analyzer
[RCS1074](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1074)
([PR](dotnet/roslynator#1768) by @​cbersch)
- Fix enum contained flags check for partial matches in
[RCS1258](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1258)
([PR](dotnet/roslynator#1740) by @​ovska)
- Fix analyzer
[RCS1146](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1146)
([PR](dotnet/roslynator#1747))
- Fix analyzer
[RCS1194](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1194)
([PR](dotnet/roslynator#1733))
- Fix analyzer
[RCS1060](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1060)
to ignore classes marked with `file` modifier
([PR](dotnet/roslynator#1777) by @​cbersch)
- Fix analyzer
[RCS1231](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1231)
([PR](dotnet/roslynator#1774) by @​cbersch)
- Fix analyzer
[RCS1246](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1246)
for conditional access expressions
([PR](dotnet/roslynator#1772 by @​krajek))
- [CLI] Fix `fix` command ignoring `--include` / `--exclude` file filter
([PR](dotnet/roslynator#1758) by @​hashiiiii)
- [CLI] Fix loading of projects and solutions on .NET 10 SDK
([PR](dotnet/roslynator#1783))


Commits viewable in [compare
view](dotnet/roslynator@v4.15.0...v4.16.0).
</details>

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore <dependency name> major version` will close this
group update PR and stop Dependabot creating any more for the specific
dependency's major version (unless you unignore this specific
dependency's major version or upgrade to it yourself)
- `@dependabot ignore <dependency name> minor version` will close this
group update PR and stop Dependabot creating any more for the specific
dependency's minor version (unless you unignore this specific
dependency's minor version or upgrade to it yourself)
- `@dependabot ignore <dependency name>` will close this group update PR
and stop Dependabot creating any more for the specific dependency
(unless you unignore this specific dependency or upgrade to it yourself)
- `@dependabot unignore <dependency name>` will remove all of the ignore
conditions of the specified dependency
- `@dependabot unignore <dependency name> <ignore condition>` will
remove the ignore condition of the specified dependency and ignore
conditions


</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
This was referenced Aug 10, 2026
Open
Open
Open
Open
Merged
Open
Open
Open
Open
Merged
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@josefpihrt josefpihrt josefpihrt approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Any way to suppress error code from Roslynator when it detects issues in code but runs successfully?

3 participants

@mdrybak @josefpihrt @davewichers