He-Pin opened a new pull request, #3075:
URL: https://github.com/apache/pekko/pull/3075

   ### Motivation
   
   The `Jdk9` sbt plugin was historically named for JDK 9 features (e.g., 
`java.util.concurrent.Flow`), but since Java 17 became the minimum baseline, 
JDK 9 features are part of the standard build. The plugin's 
`supportedJavaLTSVersions` only contains `"21"` — it exclusively manages JDK 
21+ source directories (virtual thread tests). The misleading name caused 
confusion for developers.
   
   ### Modification
   
   - **Renamed** `Jdk9` → `Jdk21`, `CompileJdk9` → `CompileJdk21`, `TestJdk9` → 
`TestJdk21`
   - **Renamed files**: `Jdk9.scala` → `Jdk21.scala`, 
`CopyrightHeaderForJdk9.scala` → `CopyrightHeaderForJdk21.scala`, 
`ScalaFixForJdk9Plugin.scala` → `ScalaFixForJdk21Plugin.scala`
   - **Renamed** `PekkoDevelocityJdk9TestSettingsPlugin` → 
`PekkoDevelocityJdk21TestSettingsPlugin`
   - **Updated** all references in `build.sbt`, `project/*.scala`, `AGENTS.md`
   - **Updated** CI workflows: `build-test-prValidation.yml`, 
`dependency-graph.yml`, `generate-doc-check.yml`
   - **Cleaned up** stale `jdk9Only` labels in `TestExtras.scala` (Flow tests 
are in regular source dirs since Java 17 is baseline)
   - **Updated** outdated comments: `"requires jdk9+"` → `"project baseline is 
Java 17"`
   - **Updated** `CONTRIBUTING.md`: `"JDK > 10"` → `"JDK >= 17"`
   - **Cleaned up** stale `"jdk9+"` references in docs test files and 
`FlightRecording.scala`
   
   ### Result
   
   Build plugin names now accurately reflect their function (JDK 21 virtual 
thread tests). Developers and CI workflows use the clearer `TestJdk21` 
configuration name. Stale JDK 9 references that no longer apply with Java 17 
baseline are removed.
   
   ### Tests
   
   - `sbt "actor-tests / compile"` — success
   - `sbt "actor-tests / TestJdk21 / compile"` — success (3 Scala + 2 Java JDK 
21 sources compiled to `TestJdk21-classes`)
   - `sbt "headerCreateAll"` — success
   - `git diff --check` — clean
   
   ### References
   
   None - build infrastructure cleanup


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to