Skip to content

Instantly share code, notes, and snippets.

Embed
What would you like to do?
Scala 2.13.3 draft release notes
@sjrd

This comment has been minimized.

Copy link

@sjrd 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

This comment has been minimized.

Copy link
Owner Author

@SethTisue 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

This comment has been minimized.

Copy link

@sjrd 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
You can’t perform that action at this time.