Skip to content

Why scala 3 tabs code #2510

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 6 commits into from
Sep 15, 2022
Merged

Why scala 3 tabs code #2510

merged 6 commits into from
Sep 15, 2022

Conversation

benluo
Copy link
Contributor

@benluo benluo commented Sep 2, 2022

add tabs for code

@bishabosha
Copy link
Member

bishabosha commented Sep 13, 2022

Im not so sure about adding translations until the book is stable, which could be another few months from now. i.e. the content might change a lot in the English version. What do you think?

@bishabosha
Copy link
Member

@jxnu-liguobin there are many files changed in this PR that don't match just the "why scala 3" page, is that intentional?

@jxnu-liguobin
Copy link
Member

@jxnu-liguobin there are many files changed in this PR that don't match just the "why scala 3" page, is that intentional?

In fact, I don't know why this PR is exists. The new commits seem for other issues. Because this PR is based on a PR that has not yet been merged.

@SethTisue
Copy link
Member

needs rebase now that #2394 is merged

@jxnu-liguobin
Copy link
Member

jxnu-liguobin commented Sep 15, 2022

needs rebase now that #2394 is merged

@benluo Please rebase it, then can be reviewd

@bishabosha bishabosha force-pushed the why-scala-3-tabs-code branch from b9a643e to 1360cae Compare September 15, 2022 08:08
@bishabosha
Copy link
Member

I just did the rebase - I see now it really is just adding code tabs, so ill review

@bishabosha bishabosha closed this Sep 15, 2022
@bishabosha bishabosha reopened this Sep 15, 2022
@bishabosha bishabosha self-requested a review September 15, 2022 08:10
@bishabosha bishabosha self-assigned this Sep 15, 2022
benluo and others added 4 commits September 15, 2022 16:46
Co-authored-by: Jamie Thompson <bishbashboshjt@gmail.com>
Co-authored-by: Jamie Thompson <bishbashboshjt@gmail.com>
Co-authored-by: Jamie Thompson <bishbashboshjt@gmail.com>
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 a lot!

@bishabosha bishabosha merged commit 05ad391 into scala:main Sep 15, 2022
@benluo benluo deleted the why-scala-3-tabs-code branch September 21, 2022 02:40
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.

4 participants