Skip to content

small changes throughout release notes #233

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

Merged
merged 2 commits into from
Nov 7, 2016

Conversation

SethTisue
Copy link
Member

@SethTisue SethTisue commented Nov 6, 2016

typos, wording changes, formatting changes, some reordering towards the top

review by @adriaanm

if merged, I'll take care of propagating the changes to the blog post and the GitHub milestone

this is a lot easier to review with git show --word-diff

typos, wording changes, formatting changes
Copy link
Member

@lrytz lrytz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!


* compile under `-Xsource:2.11`
* use an explicit call to the conversion method
* disqualify the type from being a SAM (e.g. by adding a second abstract method).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to check if this will work with scala-lang's markdown parser.. the rest of the file uses a different style for bullets.

@@ -1,93 +1,120 @@
We are very happy to announce the availability of Scala 2.12.0!

## Headline features

The Scala 2.12 compiler has been completely overhauled to make use of the new VM features available in Java 8:

- A trait [compiles directly to an interface](#trait-compiles-to-an-interface) with default methods. This improves binary compatibility and Java interoperability.
Copy link
Contributor

@adriaanm adriaanm Nov 7, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This link will need to be updated due to change to header below.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#traits-compile-to-interfaces

Copy link
Contributor

@adriaanm adriaanm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very very nice! Needs small fix for broken link.

@@ -1,93 +1,120 @@
We are very happy to announce the availability of Scala 2.12.0!

## Headline features

The Scala 2.12 compiler has been completely overhauled to make use of the new VM features available in Java 8:

- A trait [compiles directly to an interface](#trait-compiles-to-an-interface) with default methods. This improves binary compatibility and Java interoperability.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#traits-compile-to-interfaces

@adriaanm
Copy link
Contributor

adriaanm commented Nov 7, 2016

While you're at it, could you add a reference to the removal of the rewrite from withFilter -> filter done scala/scala#5252? (https://gitter.im/scala/contributors?at=58209308df5ae9664527672a)

@SethTisue SethTisue dismissed adriaanm’s stale review November 7, 2016 23:14

made requested changes

@SethTisue SethTisue merged commit fca73d0 into scala:2.12.x Nov 7, 2016
@SethTisue SethTisue deleted the release-notes-revisions branch November 7, 2016 23:15
SethTisue added a commit to scala/scala-lang that referenced this pull request Nov 7, 2016
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

Successfully merging this pull request may close these issues.

3 participants