Skip to content
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

Modify rule S1075: Add more exceptions (SONARJAVA-5149) #4570

Merged
merged 4 commits into from
Dec 16, 2024

Conversation

erwan-serandour
Copy link
Contributor

Review

A dedicated reviewer checked the rule description successfully for:

  • logical errors and incorrect information
  • information gaps and missing content
  • text style and tone
  • PR summary and labels follow the guidelines

@dorian-burihabwa-sonarsource dorian-burihabwa-sonarsource changed the title Modify rule S1075: Add more exceptions Modify rule S1075: Add more exceptions SONARJAVA-5149 Dec 11, 2024
@dorian-burihabwa-sonarsource dorian-burihabwa-sonarsource changed the title Modify rule S1075: Add more exceptions SONARJAVA-5149 Modify rule S1075: Add more exceptions (SONARJAVA-5149) Dec 11, 2024
@@ -1,5 +1,13 @@
include::../description.adoc[]

=== Exceptions

This rules doesn't raise an issue when:

Choose a reason for hiding this comment

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

Suggested change
This rules doesn't raise an issue when:
This rule does not raise an issue when:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍


This rules doesn't raise an issue when:

* A relative URI is declared in a static final constant and the URI contains at most two parts.

Choose a reason for hiding this comment

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

It might be good to provide an example if we do not already have one in the samples

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍

Comment on lines 8 to 9
* An URI is declared in a static final constant and the constant is used in an annotation.
* An URI is declared in a variable annotated with an annotation.

Choose a reason for hiding this comment

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

Suggested change
* An URI is declared in a static final constant and the constant is used in an annotation.
* An URI is declared in a variable annotated with an annotation.
* A URI is declared in a static final constant and the constant is used in an annotation.
* A URI is declared in a variable annotated with an annotation.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍

Copy link

Quality Gate passed Quality Gate passed for 'rspec-tools'

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

Copy link

Quality Gate passed Quality Gate passed for 'rspec-frontend'

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@erwan-serandour erwan-serandour merged commit 70e2b8e into master Dec 16, 2024
9 checks passed
@erwan-serandour erwan-serandour deleted the erwan/modify-S1075 branch December 16, 2024 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants