Skip to content
This repository was archived by the owner on Dec 11, 2020. It is now read-only.

Bump Roslynator.Analyzers from 2.3.0 to 3.0.0#391

Merged
dependabot[bot] merged 1 commit intomasterfrom
dependabot/nuget/Roslynator.Analyzers-3.0.0
Sep 11, 2020
Merged

Bump Roslynator.Analyzers from 2.3.0 to 3.0.0#391
dependabot[bot] merged 1 commit intomasterfrom
dependabot/nuget/Roslynator.Analyzers-3.0.0

Conversation

@dependabot
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Sep 8, 2020

Bumps Roslynator.Analyzers from 2.3.0 to 3.0.0.

Release notes

Sourced from Roslynator.Analyzers's releases.

v3.0.0

v2.9.0

  • Switch to Roslyn 3.x libraries
  • Add Directory.Build.props file
  • Add open configuration commands to Command Palette (VS Code) (PR)

Bug Fixes

  • Fix key duplication/handle camel case names in omnisharp.json (PR)
  • Use prefix unary operator instead of postfix unary operator (RCS1089) (issue)
  • Cast of this to its interface cannot be null (RCS1202) (issue)
  • Do not remove braces in switch section if it contains 'using variable' (RCS1031) (issue)

New Analyzers

  • RCS1242 (DoNotPassNonReadOnlyStructByReadOnlyReference).
  • RCS1243 (DuplicateWordInComment).
  • RCS1244 (SimplifyDefaultExpression).
  • RCS1245 (SimplifyConditionalExpression2) (issue).

Analyzers

  • Disable analyzer RCS1057 by default (issue).
  • Merge analyzer RCS1156 with RCS1113 (issue).
    • x == "" should be replaced with string.IsNullOrEmpty(x)
  • Improve analyzer RCS1215 (commit).
    • x == double.NaN should be replaced with double.IsNaN(x)
  • Enable RCS1169 and RCS1170 if the type is read-only struct (commit).
  • Improve analyzer RCS1077 (commit).
    • x.OrderBy(y => y).Reverse() can be simplified to x.OrderByDescending(y => y)
    • x.SelectMany(y => y).Count() can be simplified to x.Sum(y => y.Count) if x has Count or Length property
  • Improve analyzer RCS1161 - Declare explicit enum value using << operator (commit).
  • Improve analyzer RCS1036 - remove empty line between documentation comment and declaration (commit).
  • Improve analyzer RCS1037 - remove trailing white-space from documentation comment (commit).
  • Improve analyzer RCS1143 (commit)
    • x?.M() ?? default(int?) can be simplified to x?.M() if x is a nullable struct.
  • Improve analyzer RCS1206 (commit)
    • (x != null) ? x.M() : default(int?) can be simplified to x?.M() if x is a nullable struct.

v2.3.1

  • Last release of package Roslynator.Analyzers (2.3.0) that references Roslyn 2.x (VS 2017)
Changelog

Sourced from Roslynator.Analyzers's changelog.

3.0.0 (2020-06-16)

New Analyzers

  • RCS0048 (Remove newlines from initializer with single-line expression).
  • RCS0049 (Add empty line after top comment).
  • RCS0050 (Add empty line before top declaration).
  • RCS0051 (Add newline between closing brace and 'while' keyword (or vice versa)).
  • RCS1246 (Use element access).

New Refactorings

  • RR0214 (Convert 'switch' expression to 'switch' statement).

2.9.0 (2020-03-13)

  • Switch to Roslyn 3.x libraries
  • Add Directory.Build.props file
  • Add open configuration commands to Command Palette (VS Code) (PR)

Bug Fixes

  • Fix key duplication/handle camel case names in omnisharp.json (PR)
  • Use prefix unary operator instead of postfix unary operator (RCS1089) (issue)
  • Cast of this to its interface cannot be null (RCS1202) (issue)
  • Do not remove braces in switch section if it contains 'using variable' (RCS1031) (issue)

New Analyzers

  • RCS1242 (DoNotPassNonReadOnlyStructByReadOnlyReference).
  • RCS1243 (DuplicateWordInComment).
  • RCS1244 (SimplifyDefaultExpression).
  • RCS1245 (SimplifyConditionalExpression2) (issue).

Analyzers

Commits
  • 8fadfa9 Update version to 3.0.0.0
  • fcab5c7 Update documentation to 3.0
  • 62e770e Add missing using directives
  • 57d58f1 Remove temporary code
  • cf9e35a Remove preprocessor directives for NETCOREAPP3_1
  • 36b7468 Fix breaking change after update to .NET 4.8
  • 2719726 Update AnalyzerOptions.md
  • b0c9f7f Change target framework from net472 to net48
  • d0fe56d Change title
  • e09ad0e Update package references
  • Additional commits viewable in compare view

Dependabot compatibility score

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 @rsg-bot.


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

Bumps [Roslynator.Analyzers](https://github.com/JosefPihrt/Roslynator) from 2.3.0 to 3.0.0.
- [Release notes](https://github.com/JosefPihrt/Roslynator/releases)
- [Changelog](https://github.com/JosefPihrt/Roslynator/blob/master/ChangeLog.md)
- [Commits](dotnet/roslynator@v2.3.0...v3.0.0)

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added .NET 📦 dependencies Pull requests that update a dependency file labels Sep 8, 2020
@github-actions github-actions bot added this to the v7.0.0 milestone Sep 8, 2020
@codecov
Copy link

codecov bot commented Sep 8, 2020

Codecov Report

Merging #391 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #391   +/-   ##
=======================================
  Coverage   12.79%   12.79%           
=======================================
  Files          18       18           
  Lines         422      422           
  Branches       49       49           
=======================================
  Hits           54       54           
  Misses        368      368           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 270184d...4980d9d. Read the comment docs.

@github-actions github-actions bot removed the .NET label Sep 10, 2020
@david-driscoll david-driscoll added the :shipit: merge Shipit! label Sep 11, 2020
@rsg-bot
Copy link

rsg-bot commented Sep 11, 2020

@dependabot squash and merge

@dependabot dependabot bot merged commit 4aff31a into master Sep 11, 2020
@dependabot dependabot bot deleted the dependabot/nuget/Roslynator.Analyzers-3.0.0 branch September 11, 2020 05:30
@mergify mergify bot removed the :shipit: merge Shipit! label Sep 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

📦 dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants