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

class level granularity #137

Closed
grahamdyson opened this issue Jul 7, 2020 · 1 comment
Closed

class level granularity #137

grahamdyson opened this issue Jul 7, 2020 · 1 comment

Comments

@grahamdyson
Copy link
Member

Currently the C# analyzer represents items and dependencies at the member level. Member ordering in C# is rarely consistent with dependencies within the type e.g. top down or bottom up. If there are is some consistency in member order its usually by implementation i.e. grouping constructors, fields, properties and methods together.

Showing dependency counts between members is likely to distract from the higher level class dependencies when using Eunice for the first time on an existing code bases. Class level granularity will mean dependencies can't be listed as precisely when investigating counts. So even if users aren't interested in member order, the existing member level granularity will still be a useful option.

Having more granular items and dependencies also has a performance penalty when rendering and interacting in Eunice.

@grahamdyson
Copy link
Member Author

released in C# 1.9.0

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

1 participant