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.

Sign up for GitHub

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

Jump to bottom

refactor: expose the DefaultReporter #226

Closed
wants to merge 2 commits into from
Closed

refactor: expose the DefaultReporter #226

wants to merge 2 commits into from

Commits on Jun 30, 2020

  1. refactor: expose the DefaultReporter 

    I am trying to stop go-cmp from calling fmt.Stringer, as this in some
cases makes the output worse instead of better.

The DefaultReporter implementation is quite customizable, but this
functionality is not exposed to the user.

While users can register custom exporters, they would then need to
re-implement the entire DefaultExporter, even though changing a single
bool in it's config would suffice.

This PR exposes the DefaultReporter implementation so it can be
customized and used outside of this package
    @sh0rez
    sh0rez committed Jun 30, 2020
    Configuration menu
    Copy the full SHA
    2f12b4d View commit details
    Browse the repository at this point in the history

Commits on Jul 21, 2020

  1. Merge branch 'master' into expose-exporter

    @sh0rez
    sh0rez committed Jul 21, 2020
    Configuration menu
    Copy the full SHA
    f86bb0a View commit details
    Browse the repository at this point in the history