Skip to content

docs: Announce core decision about JDK #1763

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.

Sign up for GitHub

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
Merged
merged 7 commits into from
Mar 5, 2025
Merged

docs: Announce core decision about JDK #1763

merged 7 commits into from
Mar 5, 2025

Conversation

tgodzik
Copy link
Contributor

@tgodzik tgodzik commented Feb 27, 2025

No description provided.

@tgodzik tgodzik requested review from lrytz and SethTisue February 27, 2025 16:25
SethTisue
SethTisue reviewed Feb 27, 2025
View reviewed changes
@SethTisue
Copy link
Member

SethTisue commented Feb 27, 2025 •
edited
Loading

It feels to me that something is missing here, namely, a summary of our reasoning, based on the input received.

UPDATE: This has been addressed.

@tgodzik tgodzik force-pushed the jdk-announcement branch 2 times, most recently from bc41d8c to 88ebb2c Compare February 28, 2025 20:16
@tgodzik tgodzik requested a review from SethTisue February 28, 2025 20:16
Gedochao
Gedochao reviewed Mar 3, 2025
View reviewed changes
@SethTisue
Copy link
Member

@adpi2 wrote:

Could you move this to the blog category?
We discussed these categories with Darja a few weeks ago, and we concluded that announcement should only be used for release announcements. The Scala Highlights should be in highlights and every thing else should be in the blog category.

Done. (And, leaving this comment to be especially sure people see it and we remember next time.)

@tgodzik tgodzik merged commit d6fa0bc into scala:main Mar 5, 2025
1 check passed
@tgodzik
Copy link
Contributor Author

tgodzik commented Mar 5, 2025

Thanks @SethTisue for finishing and the help with the blogpost!

@tgodzik tgodzik deleted the jdk-announcement branch March 5, 2025 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adpi2 adpi2 adpi2 left review comments

@SethTisue SethTisue SethTisue approved these changes

@sjrd sjrd sjrd approved these changes

@Gedochao Gedochao Gedochao approved these changes

@lrytz lrytz Awaiting requested review from lrytz

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@tgodzik @SethTisue @sjrd @adpi2 @Gedochao