Skip to content

feat: initialize cognitive complexity #174

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
Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

feat: initialize cognitive complexity #174

wants to merge 4 commits into from

Conversation

junners
Copy link
Contributor

@junners junners commented Feb 6, 2025 •
edited
Loading

WIP

Rule Id Description Reference
Cognitive Complexity Much like code, flows can be hard to understand if it's trying to achieve multiple behaviors, the grander scheme of things is to implement SRP + Flow Ordering tbd

Todo

  • feature flag tree node extraction. may cause stack exceptions
  • Profiling for memory on stack
  • Support for loops
  • Update docs
  • Complete unfinished tests

@RubenHalman
Copy link
Contributor

Hi @junners this is awesome! I had some stale development related to Cyclomatic Complexity, which is slightly different, but there might be some overlap. I created a PR for it just in case. #175

@junners
Copy link
Contributor Author

junners commented Feb 13, 2025

@RubenHalman - I'll change this to an actual cognitive complexity patterned here https://www.youtube.com/watch?v=x5V2nvxco90

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@junners @RubenHalman