Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions src/main/java/org/javademos/init/Java16.java
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,10 @@ public static ArrayList<IDemo> getDemos() {
java16DemoPool.add(new VectorAPIDemo());
// JEP 347
java16DemoPool.add(new Cpp14FeaturesDemo());
// Inside the list/map initialization in the JDK 16 helper file
Copy link
Owner

Choose a reason for hiding this comment

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

Here only // JEP 375 comment should be placed, as for other demos

// ... other demos ...
demos.add(new org.javademos.java16.jep357.MercurialToGitMigration()); // JEP 357
Copy link
Owner

Choose a reason for hiding this comment

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

For consistency with other demos, do not use absolute import path here. Use import at the begining of the file and here only class name.

// ... other demos ...
// JEP 380
java16DemoPool.add(new UnixDomainSocketsDemo());
// JEP 386
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
package org.javademos.java16.jep357;

import org.javademos.commons.IDemo;

/**
* ## JEP 357: Migrate from Mercurial to Git
Copy link
Owner

Choose a reason for hiding this comment

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

Please, change to Markdown format as in other demos. Check IDemo.java for reference.

*
* ### History
* * JDK 16: Delivered
*
* ### Links
* * [JEP 357](https://openjdk.org/jeps/357)
* * [JEP 369: Migrate to GitHub](https://openjdk.org/jeps/369)
*
* This JEP was purely an infrastructure change for the OpenJDK community.
* There are no Java language features or APIs to demonstrate.
*/
public class MercurialToGitMigration implements IDemo {

@Override
public void demo() {
info(357);

System.out.println("This JEP was an infrastructure project to migrate the OpenJDK Community's source code repositories.");
Copy link
Owner

Choose a reason for hiding this comment

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

The information are correct, but please, do not use println statements for explaining. The output would be too long if all demos be like that. Turn them to comments.

System.out.println("The migration was from Mercurial (hg) to Git.");
System.out.println();
System.out.println("Key Goals:");
System.out.println("1. Migrate all single-repository OpenJDK Projects.");
System.out.println("2. Preserve all version control history, including tags.");
System.out.println("3. Port internal tools (like 'jcheck') to work with Git.");
System.out.println();
System.out.println("This change did not introduce any new Java language features or APIs, " +
"but was a critical step for the community, leading to JEP 369 (Migrate to GitHub).");
}
}
8 changes: 8 additions & 0 deletions src/main/resources/JDK16Info.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,14 @@
"link": false,
"code": false
},
{
"no": 357,
"jdk": 16,
"name": "Migrate from Mercurial to Git",
"description": "Migrate the OpenJDK Community's source code repositories from Mercurial (hg) to Git.",
"link": false,
"code": false
},
{
"jep": 380,
"jdk": 16,
Expand Down