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

Consider merging policy and logging into main assembly #23

Open
iancooper opened this issue Jan 15, 2019 · 0 comments
Open

Consider merging policy and logging into main assembly #23

iancooper opened this issue Jan 15, 2019 · 0 comments
Labels

Comments

@iancooper
Copy link
Member

The problem with the separation of policy and logging is that most of our examples show Darker being used successfully with logging and policy attributes, but because these are not brought into scope unless we add these packages separately it can be confusing to the new user how to achieve the basic scenario.

Packaging principles would say that things that 'the unit of re-use is the unit of release', and 'keep things that change together, together'. I don't see real advantage or likelihood of these packages being released on a different schedule, so separate packages actually make this harder.

Brighter uses a single package for all these requirements, so the difference is a little confusing.

In addition ASP.NET Core has moved away from this kind of 'fine grained' package model as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant