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

Jump to bottom

docs: fix pattern matching example #2189

Closed

JurgisSi wants to merge 1 commit into scala:main from JurgisSi:pattern-matching-print

Contributor

JurgisSi commented Sep 23, 2021

Hey, I think the current comment is incorrect, since the method does not print the message.

I have another pr with comment rewording, if you feel it would be better:
#2188


          docs: fix pattern matching example

2576fac

JurgisSi mentioned this pull request

docs: reword pattern matching comment #2188

Merged

Philippus approved these changes

View reviewed changes

Member

SethTisue commented Sep 30, 2021 •

edited

I merged #2188 instead, as it seems more in keeping with functional style. Thank you!

SethTisue closed this

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