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

[apex] Merge NCSS count rules for Apex #2128

Open
3 tasks
oowekyala opened this issue Nov 24, 2019 · 0 comments
Open
3 tasks

[apex] Merge NCSS count rules for Apex #2128

oowekyala opened this issue Nov 24, 2019 · 0 comments
Labels
an:enhancement An improvement on existing features / rules
Milestone
7.x

Comments

@oowekyala
Copy link
Member

oowekyala commented Nov 24, 2019 •
edited by adangel

The Apex module still has fragmented rules for NCSS measuring:

The idea is to merge them, like was done in Java, to have an easy alternative to ExcessiveClassLength, so that #2127 can be implemented for those modules too.

  • Create NCSS metric for apex
  • Create new rule NcssCount
  • Deprecate the rules ExcessiveClassLength, NcssTypeCount, NcssMethodCount, NcssConstructorCount in favor of the new NcssCount

Similar task for plsql: #4326
@adangel adangel changed the title [core] Merge NCSS count rules for Apex and Plsql [apex][plsql] Merge NCSS count rules for Apex and Plsql Nov 25, 2019
@oowekyala oowekyala added the an:enhancement An improvement on existing features / rules label Jan 17, 2021
@adangel adangel changed the title [apex][plsql] Merge NCSS count rules for Apex and Plsql [apex] Merge NCSS count rules for Apex Jan 12, 2023
@adangel adangel added this to the 7.0.0 milestone Jan 12, 2023
@adangel adangel mentioned this issue Jan 12, 2023
Open
3 tasks
@adangel adangel modified the milestones: 7.0.0, 7.x Jan 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
an:enhancement An improvement on existing features / rules
Projects
None yet
Milestone
7.x
Development

No branches or pull requests
2 participants
@adangel @oowekyala