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.

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

Feature ideas - Adding cyclic namespace dependency check #37

Open
mesteves-mw opened this issue Jun 21, 2018 · 2 comments
Open

Feature ideas - Adding cyclic namespace dependency check #37

mesteves-mw opened this issue Jun 21, 2018 · 2 comments
Milestone

Comments

@mesteves-mw
Copy link

Hi Frenec,

Thanks for the awesome tool. One thing I would love to see implemented here is a cyclic/circular namespace dependency check with an option of warning/error/silent.

@realvizu
Copy link
Owner

Thanks for submitting it. Sounds like a useful feature.
Notes:

@realvizu realvizu added this to the v1.9 milestone Jun 24, 2018
@mesteves-mw
Copy link
Author

I am not familiar with QuickGraph but DFS is simple enough to implement instead of referencing an external library and keeps the analyzer more portable. We would expect the flagged violations to be deterministic. So ideally the graph would have to be pre-built before the code inspections and remain cached until a project rebuild occurs (or a code change for live inspections). This is going to be a challenging one.

@realvizu realvizu modified the milestones: v1.9, v2.x Feb 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants