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

Bump the analyzers group with 3 updates #122

Merged
merged 2 commits into from
May 19, 2024

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github May 12, 2024

Bumps the analyzers group with 3 updates: Roslynator.Analyzers, Roslynator.Formatting.Analyzers and SonarAnalyzer.CSharp.

Updates Roslynator.Analyzers from 4.12.2 to 4.12.3

Release notes

Sourced from Roslynator.Analyzers's releases.

v4.12.3

Fixed

Changelog

Sourced from Roslynator.Analyzers's changelog.

[4.12.3] - 2024-05-10

Fixed

Commits

Updates Roslynator.Formatting.Analyzers from 4.12.2 to 4.12.3

Release notes

Sourced from Roslynator.Formatting.Analyzers's releases.

v4.12.3

Fixed

Changelog

Sourced from Roslynator.Formatting.Analyzers's changelog.

[4.12.3] - 2024-05-10

Fixed

Commits

Updates SonarAnalyzer.CSharp from 9.24.0.89429 to 9.25.0.90414

Release notes

Sourced from SonarAnalyzer.CSharp's releases.

9.25

Hello everyone,

This release comes with seven new rules for ASP.NET core alongside some improvements. Enjoy!

New Rules

  • 9096 - [C#] New Rule S6966: Awaitable method should be used
  • 9095 - [C#] New Rule S6967: ModelState.IsValid should be called in controller actions
  • 9094 - [C#] New Rule S6964: The value type properties of a model class should be nullable or marked as "Required" to avoid under-posting.
  • 9093 - [C#] New rule S6968: Actions that return a value should be annotated with ProducesResponseTypeAttribute containing the return type
  • 9092 - [C#] New rule S6965: You should use HttpAttribute in API controller actions
  • 9091 - [C#] New rule S6962: You should pool HTTP connections with HttpClientFactory
  • 9089 - [C#] New rule S6960: Controllers should not have too many responsibilities

Bug Fixes

  • 9193 - Fix AD0001: Named Attribute Arguments in S6930

False Positive

  • 9219 - [C#] Fix S1144 FP: Getters/Setters of property with attribute are being flagged

Improvements

  • 9187 - Update RSPEC before 9.25 release
  • 9186 - [C#] Rule S6961: Implement CodeFix

Rule deprecations and deletions

  • 9175 - [VB.NET] Delete S2353: Remove deprecated rule
  • 9189 - [C#] Deprecate S6803
Commits
  • 75e3904 Update RSPEC before 9.25 release (#9246)
  • bc4c63b Fix S6966 FPs/FNs: FPs/FNs after peach validation (#9222)
  • e0dce12 New rule S6960 for C#: Controllers should not have too many responsibilities ...
  • 7e0dada Fix Razor location UTs (#9242)
  • 24ec493 Improve S6964: Include array types and nested collection types (#9239)
  • c298484 Refactor: Extract common method to check if any ancestor belongs to SyntaxKin...
  • ca0f118 Decompose SymbolHelper into dedicated extension classes (#9229)
  • c755a17 Enable razor UTs in-memory compilation (#9226)
  • 75e2d7b Repro for FN 2589: Support redundant recursive pattern null check (#9234)
  • a2cd911 Fix S1144 FP: Getters/Setters of property with attribute are being flagged (#...
  • Additional commits viewable in compare view

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 will merge this PR once CI passes on it, as requested by @piotrzajac.


Dependabot commands and options

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 merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @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

Bumps the analyzers group with 3 updates: [Roslynator.Analyzers](https://github.com/dotnet/roslynator), [Roslynator.Formatting.Analyzers](https://github.com/dotnet/roslynator) and [SonarAnalyzer.CSharp](https://github.com/SonarSource/sonar-dotnet).


Updates `Roslynator.Analyzers` from 4.12.2 to 4.12.3
- [Release notes](https://github.com/dotnet/roslynator/releases)
- [Changelog](https://github.com/dotnet/roslynator/blob/main/ChangeLog.md)
- [Commits](dotnet/roslynator@v4.12.2...v4.12.3)

Updates `Roslynator.Formatting.Analyzers` from 4.12.2 to 4.12.3
- [Release notes](https://github.com/dotnet/roslynator/releases)
- [Changelog](https://github.com/dotnet/roslynator/blob/main/ChangeLog.md)
- [Commits](dotnet/roslynator@v4.12.2...v4.12.3)

Updates `SonarAnalyzer.CSharp` from 9.24.0.89429 to 9.25.0.90414
- [Release notes](https://github.com/SonarSource/sonar-dotnet/releases)
- [Commits](SonarSource/sonar-dotnet@9.24.0.89429...9.25.0.90414)

---
updated-dependencies:
- dependency-name: Roslynator.Analyzers
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: analyzers
- dependency-name: Roslynator.Formatting.Analyzers
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: analyzers
- dependency-name: SonarAnalyzer.CSharp
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: analyzers
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added .NET Pull requests that update .net code dependencies Pull requests that update a dependency file labels May 12, 2024
@codecov-commenter
Copy link

codecov-commenter commented May 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (a1a380c) to head (33cbddc).
Report is 1 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #122   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           39        39           
  Lines          423       423           
  Branches        54        54           
=========================================
  Hits           423       423           
Flag Coverage Δ
unittests 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@piotrzajac
Copy link
Collaborator

@dependabot squash and merge

@dependabot dependabot bot merged commit 7ae817d into master May 19, 2024
19 checks passed
@dependabot dependabot bot deleted the dependabot/nuget/analyzers-7811d116b5 branch May 19, 2024 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file .NET Pull requests that update .net code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants