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

TextOutput ignores the --verbose and --quiet command-line flags #159

Closed
Gabriella439 opened this issue Sep 11, 2015 · 2 comments
Closed

Comments

@Gabriella439
Copy link
Contributor

The TextOutput class takes two arguments named verbose and quiet that both default to false:

https://github.com/scalastyle/scalastyle/blob/master/src/main/scala/org/scalastyle/Output.scala#L68

However, the only use of TextOutput does not supply these arguments:

val outputResult = new TextOutput(config).output(messages)

... so they always default to false, even if the user supplies --verbose or --quiet on the command line.

I can submit a pull request to fix this if that is okay with you.

@Gabriella439
Copy link
Contributor Author

A corollary of this issue is that the --verbose flag is actually completely ignored by scalastyle

jsirois pushed a commit to pantsbuild/pants that referenced this issue Sep 12, 2015
This adds new `--verbose` and `--quiet` options to the `compile.scalastyle` goal, which get threaded directly to scalastyle, as requested in issue #1888.

Note: `scalastyle` currently accepts but ignores the `--verbose` option (See: scalastyle/scalastyle#159).  However, I include it anyway for when that issue is eventually fixed.

Testing Done:
$ ./pants test tests/python/pants_test/tasks:scalastyle_integration # Newly added integration test passes
    $ ./pants test tests/python/pants_test:all # Unit tests pass

I'm relying on Travis for the remaining integration tests

Bugs closed: 2172

Reviewed at https://rbcommons.com/s/twitter/r/2799/
@matthewfarwell
Copy link
Member

A PR would be great.

kitbellew pushed a commit to kitbellew/scalastyle that referenced this issue Jul 15, 2021
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

No branches or pull requests

2 participants