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

optIn.annotationNewlines = true #942

Closed
olafurpg opened this Issue May 24, 2017 · 1 comment

Comments

1 participant
@olafurpg
Member

olafurpg commented May 24, 2017

// optIn.annotationNewlines = true
case class ScalafixOptions(
    @HelpMessage("Print version number and exit")
    @ExtraName("v")
    version: Boolean = false,
    @HelpMessage("If set, print out debugging inforation to stderr.")
    verbose: Boolean = false,
    @HelpMessage("File path to a .scalafix.conf configuration file.")
    @ValueDescription(".scalafix.conf")
    @ExtraName("c")
    config: Option[String] = None,
    @HelpMessage("String representing scalafix configuration")
    @ValueDescription("imports.organize=false")
    @ExtraName("c")
    configStr: Option[String] = None
)
//optIn.annotationNewlines = false (current default)
case class ScalafixOptions(
    @HelpMessage("Print version number and exit") @ExtraName("v") version: Boolean =
      false,
    @HelpMessage("If set, print out debugging inforation to stderr.") verbose: Boolean =
      false,
    @HelpMessage("File path to a .scalafix.conf configuration file.") @ValueDescription(
      ".scalafix.conf") @ExtraName("c") config: Option[String] = None,
    @HelpMessage("String representing scalafix configuration") @ValueDescription(
      "imports.organize=false") @ExtraName("c") configStr: Option[String] =
      None
)

@olafurpg olafurpg added this to the v1.0 milestone May 24, 2017

@olafurpg

This comment has been minimized.

Show comment
Hide comment
@olafurpg

olafurpg May 24, 2017

Member

I don't suspect this is a controversial change, this will also fix #936

Member

olafurpg commented May 24, 2017

I don't suspect this is a controversial change, this will also fix #936

olafurpg added a commit to olafurpg/scalafmt that referenced this issue May 27, 2017

@olafurpg olafurpg closed this in 047e04e May 27, 2017

olafurpg added a commit that referenced this issue May 27, 2017

Merge pull request #953 from olafurpg/942
Enable optIn.annotationNewlines = true by default, fixes #942
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment