Skip to content

Conversation

@splusb
Copy link
Contributor

@splusb splusb commented Mar 29, 2025

Description

This PR introduces a GitHub Actions-based CI workflow to the develop branch of the log-parser-plugin.

Key additions:

  • Adds .github/workflows/ci.yml for continuous integration
  • Builds the plugin using Java 11 and Maven
  • Runs static analysis checks via PMD
  • Ensures future changes are automatically validated on push and PR

Additionally, the plugin has been tested using the Jenkins Plugin Compat Tester (PCT) to ensure compatibility with the latest Jenkins core.


Relevant issues

None opened — this is part of general modernization effort aligned with GSoC 2025 Plugin Modernizer Improvements.


Type of change

  • New feature (adds CI functionality)
  • Bug fix
  • Breaking change
  • Documentation update

How Has This Been Tested?

✅ CI workflow triggered successfully on push and pull_request events for the develop branch
✅ Build and PMD checks passed on GitHub Actions using Java 11
✅ Verified plugin compatibility with Jenkins core 2.440 using Plugin Compat Tester

PCT command:

docker run --rm \
  -v "$PWD":/plugin \
  jenkins/plugin-compat-tester \
  -Dplugin=log-parser-plugin \
  -DreportFile=target/pct-report.xml

@splusb splusb requested a review from a team as a code owner March 29, 2025 18:32
@hypery2k
Copy link
Contributor

thanks for the PR, but why didn't you just added the PMD check to the existing build.yml besides pmd everything else is alread there

Copy link
Contributor

@hypery2k hypery2k left a comment

Choose a reason for hiding this comment

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

please add pmd to build.yml

@splusb
Copy link
Contributor Author

splusb commented Apr 1, 2025

please add pmd to build.yml

I have added PMD check to the existing build.yml

@splusb splusb closed this Apr 1, 2025
@splusb splusb reopened this Apr 1, 2025
@hypery2k hypery2k merged commit a8fbf2a into jenkinsci:develop Apr 1, 2025
19 of 27 checks passed
@hypery2k
Copy link
Contributor

hypery2k commented Apr 1, 2025

thanks

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.

2 participants