Skip to content

Update why-scala-3.md #2116

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 4 commits into from
Jul 12, 2021
Merged

Update why-scala-3.md #2116

merged 4 commits into from
Jul 12, 2021

Conversation

jakedmlee
Copy link
Contributor

a.filter(_ < 30).map(_ * 10) doesn't results in List(100, 200) but List(100, 200, 100).

a.filter(_ <  30).map(_ * 10) doesn't results in List(100, 200) but List(100, 200, 100).
xuwei-k
xuwei-k previously approved these changes Jul 10, 2021
@julienrf
Copy link
Contributor

Are the two last commits intended to be part of this PR?

@jakedmlee
Copy link
Contributor Author

Are the two last commits intended to be part of this PR?

Not related to each other. But the GraalVM link address, https://www.graalvm.org/docs/reference-manual/scala/ doesn't exist.
During building PR, html-proofer is failed because of that link. (I think so.)
Many people trying to merge their code are being failed at this time. Check it out.
So for the success of checking html-proofer I removed this address.

@jakedmlee
Copy link
Contributor Author

Who are the maintainers to have write access to approve running workflows? If you can see the button, Approve and run, put the button down.

@Philippus
Copy link
Member

I don't really see an alternative to the links either, so I'm going to merge this. We can always add better links later.

@Philippus Philippus merged commit d1db634 into scala:main Jul 12, 2021
@SethTisue
Copy link
Member

perhaps @chaitanyawaikar would like to follow up on the Graal link thing (PR was #1989)

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.

5 participants