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.

Sign up for GitHub

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

Jump to bottom

Modify rules S2259;S2583;S2589;S3518;S3655;S3959 Remove replacement rules and update description for deprecated SE rules #4207

Merged
merged 2 commits into from
Aug 29, 2024
Merged

Modify rules S2259;S2583;S2589;S3518;S3655;S3959 Remove replacement rules and update description for deprecated SE rules #4207

merged 2 commits into from
Aug 29, 2024

Conversation

johann-beleites-sonarsource
Copy link
Contributor

as the new rules are slightly different and are available in different SQ editions.

@johann-beleites-sonarsource johann-beleites-sonarsource changed the title SONARJAVA-5102 REmove replacement rules for deprecated SE rules SONARJAVA-5102 Remove replacement rules for deprecated SE rules Aug 29, 2024
@leonardo-pilastri-sonarsource leonardo-pilastri-sonarsource changed the title SONARJAVA-5102 Remove replacement rules for deprecated SE rules Modify rules S2259;S2583;S2589;S3518;S3655;S3959 Remove replacement rules for deprecated SE rules Aug 29, 2024
@johann-beleites-sonarsource
SONARJAVA-5102 Remove replacement rules
5badbfb 
as the new rules are slightly different and are available in different SQ editions.
@johann-beleites-sonarsource
Add deprecation reason to rules
8478d55
@sonarqube-next SonarQube-Next
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

@sonarqube-next SonarQube-Next
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

@johann-beleites-sonarsource johann-beleites-sonarsource changed the title Modify rules S2259;S2583;S2589;S3518;S3655;S3959 Remove replacement rules for deprecated SE rules Modify rules S2259;S2583;S2589;S3518;S3655;S3959 Remove replacement rules and update description for deprecated SE rules Aug 29, 2024
@leonardo-pilastri-sonarsource leonardo-pilastri-sonarsource merged commit d4d145e into master Aug 29, 2024
11 checks passed
@leonardo-pilastri-sonarsource leonardo-pilastri-sonarsource deleted the jb/SONARJAVA-5102 branch August 29, 2024 15:27
johann-beleites-sonarsource added a commit that referenced this pull request Sep 6, 2024
@johann-beleites-sonarsource
Revert "Modify rules S2259;S2583;S2589;S3518;S3655;S3959 Remove repla…
76a7c09 
…cement rules and update description for deprecated SE rules (#4207)"

This reverts commit d4d145e.
@johann-beleites-sonarsource johann-beleites-sonarsource mentioned this pull request Sep 6, 2024
Merged
johann-beleites-sonarsource added a commit that referenced this pull request Sep 9, 2024
@johann-beleites-sonarsource
Modify rules S2259;S2583;S2589;S3518;S3655;S3959: undo deprecation (#…
60187a8 
…4245)

* Revert "Modify rules S2259;S2583;S2589;S3518;S3655;S3959 Remove replacement rules and update description for deprecated SE rules (#4207)"

This reverts commit d4d145e.

* Revert "SONARJAVA-5102 Deprecate Java SE rules implemented by DBD (#4177)"

This reverts commit 952c1ca.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leonardo-pilastri-sonarsource leonardo-pilastri-sonarsource leonardo-pilastri-sonarsource approved these changes

Assignees
No one assigned
Labels
java
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@johann-beleites-sonarsource @leonardo-pilastri-sonarsource