-
Notifications
You must be signed in to change notification settings - Fork 656
Closed
Labels
good-first-issueGood for newcomersGood for newcomershacktoberfest-acceptedPR is approved for Hacktoberfest even if not mergedPR is approved for Hacktoberfest even if not mergedis:enhancementNew feature or requestNew feature or requestup-for-grabsThis issue is open to be worked on by anyoneThis issue is open to be worked on by anyone
Description
Mentioned in #648
Issues found: https://sonarcloud.io/project/issues?resolved=false&rules=csharpsquid%3AS3626&id=nikcio_lucenenet
When there are return statements at the end of a method, we should comment them out in the source code and leave a // LUCENENET: comment after them why they were removed. A link to a related Code Analysis rule (preferred) or SonarCloud issue is fine.
For continue statements`, we should leave these as-is and add the necessary suppression to keep this issue from being detected by code analysis.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good-first-issueGood for newcomersGood for newcomershacktoberfest-acceptedPR is approved for Hacktoberfest even if not mergedPR is approved for Hacktoberfest even if not mergedis:enhancementNew feature or requestNew feature or requestup-for-grabsThis issue is open to be worked on by anyoneThis issue is open to be worked on by anyone