Skip to content

Update implicit-conversions.md #2674

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 11 commits into from
Jan 19, 2023
Merged

Update implicit-conversions.md #2674

merged 11 commits into from
Jan 19, 2023

Conversation

oliver-brm
Copy link
Contributor

I'm not sure if it is intentional, but the Tour-of-Scala documentation does not mention how to bring implicit conversions into scope. In fact, that information is a bit hard to find anywhere else, so I thought I'll propose to add it here?

I'm not sure if it is intentional, but the Tour-of-Scala documentation does not mention how to bring implicit conversions into scope. In fact, that information is a bit hard to find anywhere else, so I thought I'll propose to add it here?
oliver-brm and others added 3 commits January 18, 2023 13:10
Co-authored-by: Jamie Thompson <bishbashboshjt@gmail.com>
Improved Scala 3 description. Thanks for all the suggestions!
Copy link
Member

@bishabosha bishabosha left a comment

Choose a reason for hiding this comment

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

very nice, thanks for contributing!

@bishabosha
Copy link
Member

I've just remembered that you wrote all this when the next section "How are implicit conversions selected" is supposed to answer this point, by linking to the scala FAQ, maybe it would be good to merge the sections

bishabosha and others added 2 commits January 19, 2023 11:24
Sorry for wasting your time, next time, I'll install the toolchain.
@oliver-brm
Copy link
Contributor Author

oliver-brm commented Jan 19, 2023

OK, I can do this. The FAQ, however, only refers to this StackOverflow article, written back in 2011. I personally would not link to it from Tour of Scala, but I can place it under a Further reading section. WDYT?

Ok, that makes sense :)

Copy link
Member

@bishabosha bishabosha left a comment

Choose a reason for hiding this comment

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

Thanks for being patient :)

@bishabosha bishabosha merged commit 35ec471 into scala:main Jan 19, 2023
@oliver-brm oliver-brm deleted the patch-1 branch January 19, 2023 19:57
@oliver-brm
Copy link
Contributor Author

Thanks for being patient :)

Thanks for being patient, too! I’m happy to have contributed. 😊

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.

2 participants