Skip to content
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

chore: Move java 21 code to a dedicated folder and including code sni… #1023

Merged
merged 2 commits into from
Jan 23, 2024

Conversation

He-Pin
Copy link
Member

@He-Pin He-Pin commented Jan 22, 2024

Motivation:
Group code in Java and Java21 tab, and as the origin is under in @@@ div { .group-java }, so It will not be visible to Scala user too.

image

Continue of #1011

@He-Pin He-Pin requested a review from pjfanning January 22, 2024 19:08
@He-Pin He-Pin requested a review from pjfanning January 22, 2024 19:43
@duobei
Copy link
Contributor

duobei commented Jan 23, 2024

That's great. So that's how it's done.

@He-Pin He-Pin requested a review from mdedetrich January 23, 2024 03:22
Copy link
Contributor

@nvollmar nvollmar left a comment

Choose a reason for hiding this comment

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

lgtm

@He-Pin He-Pin merged commit 6df4b88 into apache:main Jan 23, 2024
18 checks passed
@He-Pin He-Pin deleted the java21 branch January 23, 2024 06:37
Copy link
Member

@Roiocam Roiocam left a comment

Choose a reason for hiding this comment

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

LGTM

}
}
```
Java 21
Copy link
Member

Choose a reason for hiding this comment

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

Looks great, this is kind of like Tabs feature in docusaurus.

I often use Tabs when writing release notes at work.

I think this is a good example when we add the newest JDk-related features in the future.


// #pattern-matching

static class OptimizedActorWithJava21 extends UntypedAbstractActor {
Copy link
Member

Choose a reason for hiding this comment

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

should also add there source directory on the build script, as JDK9.scala does.

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