-
-
Notifications
You must be signed in to change notification settings - Fork 540
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
A few fixed qodana issues #2379
Conversation
Qodana for .NETIt seems all right 👌 No new problems were found according to the checks applied 💡 Qodana analysis was run in the pull request mode: only the changed files were checked View the detailed Qodana reportTo be able to view the detailed Qodana report, you can either:
To get - name: 'Qodana Scan'
uses: JetBrains/qodana-action@v2023.2.8
with:
upload-result: true Contact Qodana teamContact us at qodana-support@jetbrains.com
|
I don't see Qodana complaining about this. Where do you see this? Secondly, should we just silence that analyzer instead? |
Pretty funny thing.. I accidentally double-clicked on the |
If this cannot be automatically enforced on save across whatever IDE contributors are using, I don't think this consistency is worth pursuing, i.e. nagging contributors about. Related comment about trailing commas. |
So... if I understand this correctly: The manual enforcement of such rules are not what we want? |
d61d40b
to
a2511e5
Compare
Ignoring trailing comma complaints I left the first commit remaining, since redundant parentheses are both unnecessary and confusing sometimes |
Pull Request Test Coverage Report for Build 6531685737
💛 - Coveralls |
IMPORTANT
./build.sh --target spellcheck
or.\build.ps1 --target spellcheck
before pushing and check the good outcome