Skip to content

Instantly share code, notes, and snippets.

@SethTisue
Last active June 25, 2020 21:06
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save SethTisue/3c5c425cb884ed4b912dd97bf1d5afeb to your computer and use it in GitHub Desktop.
Save SethTisue/3c5c425cb884ed4b912dd97bf1d5afeb to your computer and use it in GitHub Desktop.
Scala 2.13.3 draft release notes
@sjrd
Copy link

sjrd commented Jun 24, 2020

@SethTisue

  • Deprecate auto-application of (non-Java-defined) methods with a single empty parameter list (#8833)
    • The PR has instructions for suppressing the warning if it is unwanted
  • Warn by default on non-matching overrides of nullary/nilary methods (#8846)

I recommend putting the instructions to silence those directly in the release notes. IME they cause lots of new warnings, so everyone will want those. The instructions in the PR are not self-explanatory and are written for a special audience (e.g., the scala incantations have --scala-pr 8833; that's not a typical command line for users).

@SethTisue
Copy link
Author

SethTisue commented Jun 24, 2020

@sjrd thanks — I revamped the instructions in the PR description. do you think that's a sufficient fix, or do you think the release notes still ought to change?

cc @dwijnand who might want to check my rewrite

@sjrd
Copy link

sjrd commented Jun 24, 2020

It's probably a sufficient fix, yes. Beyond that it's a matter of style of the release notes. It's the kind of things I spell out in release notes, but then my release notes are more section-oriented whereas the Scala release notes are more bullet-oriented, so YMMV.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment