-
Notifications
You must be signed in to change notification settings - Fork 30
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
Comments
Thanks for submitting it. Sounds like a useful feature.
|
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. |
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.
The text was updated successfully, but these errors were encountered: