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

build(deps): bump Roslynator.Analyzers from 2.3.0 to 3.0.0 #602

Merged
merged 1 commit into from
Sep 9, 2020

Conversation

dependabot-preview[bot]
Copy link
Contributor

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 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 ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language
  • @dependabot badge me will comment on this PR with code to add a "Dependabot enabled" badge to your readme

Additionally, you can set the following in your Dependabot dashboard:

  • Update frequency (including time of day and day of week)
  • Pull request limits (per update run and/or open at any time)
  • Automerge options (never/patch/minor, and dev/runtime dependencies)
  • Out-of-range updates (receive only lockfile updates, if desired)
  • Security updates (receive only security updates, if desired)

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-preview[bot] <support@dependabot.com>
@dependabot-preview dependabot-preview bot added the dependencies Pull requests that update a dependency file label Sep 9, 2020
@codecov
Copy link

codecov bot commented Sep 9, 2020

Codecov Report

Merging #602 into main will increase coverage by 0.39%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #602      +/-   ##
==========================================
+ Coverage   50.82%   51.21%   +0.39%     
==========================================
  Files          56       56              
  Lines        3343     3298      -45     
  Branches      551      551              
==========================================
- Hits         1699     1689      -10     
+ Misses       1459     1424      -35     
  Partials      185      185              
Impacted Files Coverage Δ
src/Akavache.Core/ProtectedData.cs 0.00% <0.00%> (-100.00%) ⬇️
...kavache.Core/Platforms/shared/AkavacheHttpMixin.cs 26.80% <0.00%> (-1.20%) ⬇️
...kavache.Sqlite3/Queues/OperationQueueCoalescing.cs 81.06% <0.00%> (-0.12%) ⬇️
src/Akavache.Core/MD5.cs 0.00% <0.00%> (ø)
src/Akavache.Core/Platforms/shared/MD5Managed.cs 0.00% <0.00%> (ø)
...Akavache.Core/Json/JsonDateTimeContractResolver.cs 100.00% <0.00%> (ø)
src/Akavache.Sqlite3/SQLite.cs 30.95% <0.00%> (+0.62%) ⬆️
src/Akavache.Core/BlobCache/BlobCache.cs 23.33% <0.00%> (+1.11%) ⬆️

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 19458ec...bb88b7c. Read the comment docs.

@glennawatson glennawatson merged commit 43487bf into main Sep 9, 2020
@glennawatson glennawatson deleted the dependabot/nuget/Roslynator.Analyzers-3.0.0 branch September 9, 2020 00:29
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 26, 2021
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.

1 participant