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

disable optional output formatter #179

Merged
merged 2 commits into from
Nov 5, 2018
Merged

Conversation

smoench
Copy link
Contributor

@smoench smoench commented Oct 17, 2018

Graphiz is an optional dependency and JUnit requires ext-dom

@timglabisch
Copy link
Collaborator

i like this PR but it's a BC, so i am not 100% sure.

@smoench
Copy link
Contributor Author

smoench commented Oct 17, 2018

We still have no 1.0 release and printing following disclaimer:

deptrac is alpha, not production ready.

@smoench
Copy link
Contributor Author

smoench commented Oct 19, 2018

Semver is allowing us having breaking changes in a zero-mayor-version. If there no others concerns when BC it should be merged. The documentation may have to be updated and changelog/upgrade guide could added.

@timglabisch
Copy link
Collaborator

from my point it's ok. feel free to merge :)

@smoench smoench force-pushed the disable-optional-output-formatter branch from 02cbdbc to 9a58602 Compare October 24, 2018 13:03
@smoench smoench force-pushed the disable-optional-output-formatter branch from 9a58602 to e795ca2 Compare October 24, 2018 13:07
@smoench
Copy link
Contributor Author

smoench commented Oct 24, 2018

I added another/more reliable way to enable/disable formatters by default. Reviews are welcome :-)

@smoench smoench merged commit 1c4bae9 into master Nov 5, 2018
@smoench smoench deleted the disable-optional-output-formatter branch November 5, 2018 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants