-
-
Notifications
You must be signed in to change notification settings - Fork 256
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
RCS1169 not being diffused by roslynator_unity_code_analysis.enabled = true #1418
Comments
I think it has something to do with Unity not using roslynator/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs Line 21 in c2b0479
UnityEngine.SerializeFieldAttribute , but in Unity, it's declared as UnityEngine.SerializeField
|
Yes, that's the problem. Do you have any idea why suffix |
I have no idea. Might be an old thing that remained in the engine for compatibility purpures, but then they change a lot of APIs in each major version, so I don't know why they don't change this as well (low priority?). But yeah, there's not official reason publicly available anywhere. |
I checked and asked around, the attribute has always been just |
Product and Version Used:
vscode with C# dev kit | Unity project | roslynator as nuget package (latest to date)
Steps to Reproduce:
add the following to
.editorconfig
then write a script that a field with
[SerializeField]
attribute, likeActual Behavior:
(note that
Update
did not get removed, showing that[RCS1213](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1213)
is being diffused)Expected Behavior:
The text was updated successfully, but these errors were encountered: