Skip to content

Conversation

sjrd
Copy link
Member

@sjrd sjrd commented Jul 30, 2014

No description provided.

@sjrd
Copy link
Member Author

sjrd commented Jul 30, 2014

Review by @gzm0

@sjrd
Copy link
Member Author

sjrd commented Jul 30, 2014

Possibly also @lihaoyi

@sjrd
Copy link
Member Author

sjrd commented Jul 30, 2014

And the updated develop branch in the tutorial:
https://github.com/scala-js/scalajs-tutorial/commits/develop

Copy link
Contributor

Choose a reason for hiding this comment

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

on our standard benchmarks?

Copy link
Member Author

Choose a reason for hiding this comment

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

No, I meant this standard. But I am open to suggestions for better wording.

@gzm0
Copy link
Contributor

gzm0 commented Jul 30, 2014

That's all

@gzm0
Copy link
Contributor

gzm0 commented Jul 30, 2014

develop in scalajs-tutorial LGTM

@sjrd
Copy link
Member Author

sjrd commented Jul 30, 2014

Updated.

@gzm0
Copy link
Contributor

gzm0 commented Jul 30, 2014

LGTM

@sjrd
Copy link
Member Author

sjrd commented Jul 30, 2014

Perfect timing, the publishing just finished. Updating example app and these things, and then I'll announce.

@gzm0
Copy link
Contributor

gzm0 commented Jul 30, 2014

Don't forget the APIs

@sjrd
Copy link
Member Author

sjrd commented Jul 30, 2014

No, no, I'm doing that now.

sjrd added a commit that referenced this pull request Jul 30, 2014
@sjrd sjrd merged commit 1dba349 into scala-js:master Jul 30, 2014
@sjrd sjrd deleted the release-0.5.3 branch July 30, 2014 19:09
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe this is a bit late, but I don't understand this and the following lines. Is java.net.URI one of the classes that's in javalib-ex? If it is, change it to "These additional Java classes" would make it more clear =)

Copy link
Member Author

Choose a reason for hiding this comment

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

No, it's only DataInputStream, as the sentence a few lines below says:

Currently, the only additional class is java.io.DataInputStream.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah ok then, I completely missed that when skimming.

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

Labels

None yet

3 participants