Skip to content

Neolution CodeAnalysis V3#14

Merged
neoscie merged 15 commits intomainfrom
release/v3
Jun 28, 2023
Merged

Neolution CodeAnalysis V3#14
neoscie merged 15 commits intomainfrom
release/v3

Conversation

@neoscie
Copy link
Copy Markdown
Contributor

@neoscie neoscie commented Jun 14, 2023

No description provided.

neoscie added 4 commits June 8, 2023 02:33
Remove old and IDE* rules


Move everything into the right space


Fix dependencies


Also deliver the globalconfig file


dotfiles do not seem to be supported by nuspec


corrections


Revert "Remove old and IDE* rules"

This reverts commit 65a9547.

deprecated rule


Add the exceptions back to the globalconfig


Update rules


Drop IDE0055 rule


Adjust ruleset according recommended set + our overrides


Use suggestion instead of info


Somehow CS1591 was removed along the migration


Suppress S2857

Has a confirmed FP that makes it unusable when using `nameof()` SonarSource/sonar-dotnet#6126
All explanations in the globalanalyzer file


Get ready for release


Refactor build files and update global configuration
Copy link
Copy Markdown
Collaborator

@drebrez drebrez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll test it with HBL solution

Copy link
Copy Markdown
Collaborator

@drebrez drebrez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Analyzer durations with HBL solution

without   cmdline   1 min
          pipeline  2.5 min

v2.5.8    cmdline   8 min
          pipeline  20-24 min

v3 rc0    cmdline   4.5 min
          pipeline  13.5 min

v3 rc1    cmdline   5.5 min
          pipeline  13.5 min

v3 rc2    cmdline   5.6 min
          pipeline  12 min

v3 rc4    cmdline   5.5 min
          pipeline  14 min

For me it looks good 👍🏼

@neoscie neoscie merged commit ab0a2bd into main Jun 28, 2023
@neoscie neoscie deleted the release/v3 branch June 28, 2023 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants