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

Ability to output multiple formats #530

Closed
saveman71 opened this issue Jan 18, 2024 · 0 comments · Fixed by #538
Closed

Ability to output multiple formats #530

saveman71 opened this issue Jan 18, 2024 · 0 comments · Fixed by #538

Comments

@saveman71
Copy link

saveman71 commented Jan 18, 2024

Environment

  • Elixir & Erlang/OTP versions (elixir --version): 1.15.6

  • Which version of Dialyxir are you using? (cat mix.lock | grep dialyxir): 1.4.3

Current behavior

Currently, when we run mix dialyzer --format github, it only outputs the annotations, resulting in a less practical CLI output (the first place you go when you have a failed build):

image

Each of our devs failed to see at first that there was now annotations in the diff view.

Expected behavior

Ability to run mix dialyzer --format github,dialyxir or mix dialyzer --format github --format dialyxir that would output both formats.

axelson added a commit to axelson/dialyxir that referenced this issue Jul 24, 2024
Fixes jeremyjh#530

One scenario where multiple formatters is helpful is on CI where you want to use the GitHub
formatter along with a more verbose formatter to see more about the specific failures (especially
when viewing the CI logs directly).
axelson added a commit to axelson/dialyxir that referenced this issue Jul 24, 2024
Fixes jeremyjh#530

One scenario where multiple formatters is helpful is on CI where you want to use the GitHub
formatter along with a more verbose formatter to see more about the specific failures (especially
when viewing the CI logs directly).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants