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

feat(java): Require Jenkins core 2.479.3 and Java 17. #153

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from

Conversation

gounthar
Copy link
Contributor

@gounthar gounthar commented Feb 10, 2025

Hello log-parser developers! 👋

This is an automated pull request created by the Jenkins Plugin Modernizer tool. The tool has applied the following recipes to modernize the plugin:

Upgrade to the next major parent version (5.X) requiring Jenkins 2.479 and Java 17

io.jenkins.tools.pluginmodernizer.UpgradeNextMajorParentVersion

Upgrade to the next major parent version (5.X) requiring Jenkins 2.479 and Java 17.

Why Upgrade to Java 17 and Jenkins 2.479.x?

  • Embrace Java 17 LTS Stability: Benefit from long-term support with modern language features that improve development practice and plugin performance.

  • Harness Jenkins 2.479.x Innovations: Stay aligned with the latest features and stability improvements, ensuring smooth integration and future-proofing.

  • Enhance Security: Protect your plugin with up-to-date security fixes from both Java 17 and Jenkins core improvements.

  • Align with the Community: Keep pace with ecosystem shifts towards Java 17, ensuring compatibility and expanding your plugin's user base.

  • Enjoy a Better Developer Experience: Make the most of advanced tooling support and simplified dependency management with Java 17's enhancements.

Removing developers Tag from pom.xml

Jenkins no longer requires the developers tag in pom.xml, as the developers section was traditionally used to list individuals responsible for the plugin.
Instead, Jenkins now uses the Repository Permission Updater (RPU) to manage permissions and developer information.

Benefits of Removing developers Tag:
  • Simplification: Removes unnecessary metadata from the pom.xml, resulting in a cleaner and more maintainable file.
  • Consistency: Centralizes developer information management through the RPU, minimizing discrepancies.
  • Security: Utilizes the RPU's controlled permission management, enhancing the security of artifact deployments.

Removing the developers tag aligns with modern Jenkins infrastructure standards and prevents outdated or redundant developer information from being included in plugin metadata.

Summary

By upgrading, you'll be positioning your plugin at the forefront of performance, security, and user satisfaction. We encourage you to explore these updates and provide feedback. Let's continue to build a robust Jenkins ecosystem together!

@gounthar gounthar requested a review from a team as a code owner February 10, 2025 11:42
@gounthar gounthar force-pushed the plugin-modernizer/upgradenextmajorparentversion branch 3 times, most recently from fd15bbe to 0bd4f5f Compare February 10, 2025 16:33
@gounthar gounthar force-pushed the plugin-modernizer/upgradenextmajorparentversion branch from 0bd4f5f to dbfe040 Compare February 10, 2025 17:26
@gounthar gounthar force-pushed the plugin-modernizer/upgradenextmajorparentversion branch from dbfe040 to 70051d7 Compare February 15, 2025 19:37
@gounthar
Copy link
Contributor Author

The check failure is due to the maven build still using jdk11. 🤷
This builds fine on my machine and on the infra.

The build could not read 1 project -> [Help 1]
Error:    
Error:    The project org.jenkins-ci.plugins:log-parser:2.3.8-SNAPSHOT (/home/runner/work/log-parser-plugin/log-parser-plugin/pom.xml) has 1 error
Error:      Unknown packaging: hpi @ line 13, column 16
Error:  
Error:  To see the full stack trace of the errors, re-run Maven with the -e switch.
Error:  Re-run Maven using the -X switch to enable full debug logging.
Error:  
Error:  For more information about the errors and possible solutions, please read the following articles:
Error:  [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
Error: Process completed with exit code 1.

@gounthar gounthar force-pushed the plugin-modernizer/upgradenextmajorparentversion branch from 850d572 to d2ca872 Compare February 25, 2025 08:11
@gounthar gounthar changed the title feat(java): Require Jenkins core 2.479.1 and Java 17. feat(java): Require Jenkins core 2.479.3 and Java 17. Feb 25, 2025
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.

1 participant