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

Fix/issue 103 reduce cognitive complexity of AnnotationChangeRefactor#removeMembersFromAnnotation #106

Merged

RadikalJin merged 2 commits into alfasoftware:main from Dafodils:main

Jun 22, 2022

Contributor

Dafodils commented Jun 18, 2022

Fixes #103

Changes proposed in this PR:

isolated the namesForMembersToRemove being empty condition in order to reduce nesting level.

Dafodils added 2 commits

June 18, 2022 14:16


           first modif idea to reduce cognitive complexity

3c254a7


          format

32078e1

Member

RadikalJin commented Jun 22, 2022

Looks great, thank you @Dafodils! Merging now.

RadikalJin merged commit 4e14d17 into alfasoftware:main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet