Partition options by function: -V verbose, -W warnings, preserving current aliases #7908
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add standard option `-Werror` with `-Xfatal-warnings` alias. As with javac, it appears on the `-help` output. Rename `-Ywarn-foo-bar` as `-Wfoo-bar`, with the previous name as a abbreviation or alias. `-W` shows warnings, including all the `-Xlint` and `-Wunused`. Use underscore to mean all phases Move option note to bottom of message -V for verbose nee debug options Former flags future and experimental are more former. Move output options to -V. Options which modify output remain -Y. -Yprint-trees:text,compact,format,text+format replaces -Yshow-trees-stringified, et al. Reporter says -Werror Briefer choices list Briefer multichoices list Tweak expressions More -V
Also remove dependency on jackson for yaml output.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
gslowikowski
added a commit
to scoverage/scalac-scoverage-plugin
that referenced
this pull request
Apr 28, 2019
- upgrade Scala version from 2.13.0-M5 to 2.13.0-RC1 in `build.sbt` and `.travis.yml` - upgrade scalatest version from 3.0.6-SNAP4 to 3.0.8-RC2 - upgrade stala-xml version from 1.1.1 to 1.2.0 Some fixes in tests were required: - 's' and 'raw' string interpolators are now implemented as macros and macros are not instrumented; 's"Hello, $name"' expression was changed to explicit strings concatenation '"Hello, " + name' so the number of instrumented statements is the same in all Scala versions see: scala/scala@0cebcdf#diff-648e8f7f409d844c3c99b2d5a0d0dfe5R94 scala/scala#7779 - special symbol for all compilation phases was changed from "all" to "_" see: scala/scala@4d29431#diff-d686e8e67add4d42682f22193d20494bR1246 scala/scala#7908
Member
|
@som-snytt would you mind editing the PR title and description to indicate that more options are affected than just |
som-snytt
changed the title
Member
|
I've added some bullets to the PR description with some specific changes I personally have found especially notable. We can add more. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Especially notable changes include:
-Werroris recommended over-Xfatal-warningsBreaking changes include:
-Xprint:allis no longer accepted; use-Vprint:_-warn-optionis no longer accepted; use-WoptionThis is a rebase of #7686