stride icon indicating copy to clipboard operation
stride copied to clipboard

remove stylecop

Open IXLLEGACYIXL opened this issue 1 year ago • 4 comments

PR Details

Removes Stylecop from stride completely and the removal of analyzers

Description

Style Cop gets currently added to the projects, then it gets immediately removed from the analyzers. if style Cop is enabled again it would generate 22k warnings so it has to stay disabled.

Related Issue

(https://github.com/stride3d/stride/pull/2104) Discussion here

Motivation and Context

Its currently useless as it gets added and then removed.

Types of changes

  • [x] Docs change / refactoring / dependency upgrade
  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • [ ] My change requires a change to the documentation.
  • [ ] I have added tests to cover my changes.
  • [x] All new and existing tests passed.
  • [x] I have built and run the editor to try this change out.

IXLLEGACYIXL avatar Jan 13 '24 23:01 IXLLEGACYIXL

Added context for Xen2; removing DisableAnalyzersForStrideBuild is done to allow our analyzers and other third party analyzers from nuget to work by default instead of having to whitelist them. No other use or reference of StrideEnableCodeAnalysis were found in the code base.

Eideren avatar Jan 14 '24 07:01 Eideren

https://gist.github.com/IXLLEGACYIXL/3c08397b2a6a8d25b2d08c8091fa044c doesnt build anymore after branch update

IXLLEGACYIXL avatar Jan 22 '24 21:01 IXLLEGACYIXL

works to build with git clean -xfd and dotnet restore

IXLLEGACYIXL avatar Jan 23 '24 13:01 IXLLEGACYIXL

I would like a confirmation from @xen2 that it's ok to get rid of style cop. Beside that, LGTM.

Kryptos-FR avatar Jan 23 '24 14:01 Kryptos-FR

Sorry for not answering earlier. I started the D3D11 teamcity build a few days ago and it passed, LGTM!

xen2 avatar Feb 23 '24 08:02 xen2