Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[apex] New rule NcssCount to replace old Ncss*Count rules #6198

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
Merged
adangel merged 19 commits into from
Nov 27, 2025
Merged

[apex] New rule NcssCount to replace old Ncss*Count rules #6198

adangel merged 19 commits into from
Nov 27, 2025

Conversation

@adangel
Copy link
Member

@adangel adangel commented Oct 31, 2025 •
edited
Loading

Describe the PR

  • Implemented the metric Ncss for Apex as well, similar to the Java implementation
  • new rule NcssCount
  • deprecate the old rules

Related issues

Ready?

  • Added unit tests for fixed bug/feature
  • Passing all unit tests
  • Complete build ./mvnw clean verify passes (checked automatically by github actions)
  • Added (in-code) documentation (if needed)

@adangel adangel added this to the 7.19.0 milestone Oct 31, 2025
@adangel adangel added a:new-rule Proposal to add a new built-in rule is:deprecation The main focus is deprecating public APIs or rules, eg to make them internal, or removing them labels Oct 31, 2025
@pmd-actions-helper
Copy link
Contributor

pmd-actions-helper bot commented Oct 31, 2025 •
edited
Loading

Documentation Preview

Compared to main:
This changeset changes 0 violations,
introduces 2 new violations, 0 new errors and 0 new configuration errors,
removes 6 violations, 0 errors and 0 configuration errors.

Regression Tester Report

(comment created at 2025-11-13 15:07:51+00:00 for 50b368f)

@adangel adangel mentioned this pull request Nov 13, 2025
Merged
10 tasks
@adangel adangel self-assigned this Nov 27, 2025
@adangel
Copy link
Member Author

adangel commented Nov 27, 2025

I'm going to merge this for 7.19.0, assuming this is ok.

@adangel adangel merged commit 54b9ac9 into pmd:main Nov 27, 2025
12 checks passed
@adangel adangel deleted the apex/issue-2128-ncss branch November 27, 2025 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@adangel adangel

Labels

a:new-rule Proposal to add a new built-in rule is:deprecation The main focus is deprecating public APIs or rules, eg to make them internal, or removing them

Projects

None yet

Milestone

7.19.0

Development

Successfully merging this pull request may close these issues.

[apex] Merge NCSS count rules for Apex

1 participant

@adangel