Skip to content

Upgrade Maven compiler source and target to 1.8#39

Open
namastepritish wants to merge 1 commit into
NotHarshhaa:masterfrom
namastepritish:patch-1
Open

Upgrade Maven compiler source and target to 1.8#39
namastepritish wants to merge 1 commit into
NotHarshhaa:masterfrom
namastepritish:patch-1

Conversation

@namastepritish

@namastepritish namastepritish commented Jul 6, 2026

Copy link
Copy Markdown

Hi,

While testing DevOps-Project-05 with the latest Jenkins Docker image and newer JDK versions, I noticed that the Maven build fails because the project is still configured with Java 7 source/target compatibility.

Error observed:

"Source option 7 is no longer supported. Use 8 or later."

Modern JDK versions no longer support compiling with source/target 1.7, so I have updated the Maven compiler configuration from Java 1.7 to Java 1.8.

Changes:

  • Updated maven.compiler.source from 1.7 β†’ 1.8
  • Updated maven.compiler.target from 1.7 β†’ 1.8

This keeps the project compatible with newer Jenkins + Docker environments while maintaining backward compatibility.

Tested with:

  • Jenkins running in Docker
  • Maven build: mvn clean install
  • OpenJDK newer runtime

Build passes successfully after this change.

Thanks for maintaining this helpful DevOps learning repository!

Summary by CodeRabbit

  • Chores
    • Updated the project to compile with Java 8 instead of Java 7.
    • This helps keep the app aligned with newer Java environments and builds.

Hi,

While testing DevOps-Project-05 with the latest Jenkins Docker image and newer JDK versions, I noticed that the Maven build fails because the project is still configured with Java 7 source/target compatibility.

Error observed:

"Source option 7 is no longer supported. Use 8 or later."

Modern JDK versions no longer support compiling with source/target 1.7, so I have updated the Maven compiler configuration from Java 1.7 to Java 1.8.

Changes:
- Updated maven.compiler.source from 1.7 β†’ 1.8
- Updated maven.compiler.target from 1.7 β†’ 1.8

This keeps the project compatible with newer Jenkins + Docker environments while maintaining backward compatibility.

Tested with:
- Jenkins running in Docker
- Maven build: mvn clean install
- OpenJDK newer runtime

Build passes successfully after this change.

Thanks for maintaining this helpful DevOps learning repository!
@coderabbitai

coderabbitai Bot commented Jul 6, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. πŸŽ‰

ℹ️ Recent review info
βš™οΈ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d65efdd5-d3b4-4330-92e6-66d31feb7ff3

πŸ“₯ Commits

Reviewing files that changed from the base of the PR and between a38cc0f and 7c217da.

πŸ“’ Files selected for processing (1)
  • DevOps-Project-05/hello-world/pom.xml

πŸ“ Walkthrough

Walkthrough

The Maven compiler plugin configuration in the hello-world pom.xml file is updated, changing both the source and target Java version from 1.7 to 1.8.

Changes

Compiler Configuration Update

Layer / File(s) Summary
Update Java compiler version
DevOps-Project-05/hello-world/pom.xml
Maven compiler plugin's <source> and <target> values changed from 1.7 to 1.8.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Related Issues: None specified

Related PRs: None specified

Suggested labels: dependencies, build

Suggested reviewers: NotHarshhaa

Poem:
A rabbit hops through old code's rows,
Bumping numbers, that's how it goes.
1.7 fades, 1.8 takes its place,
A tiny tweak, a compiler's grace.
Twitching whiskers, the build now flows! 🐰

πŸš₯ Pre-merge checks | βœ… 5
βœ… Passed checks (5 passed)
Check name Status Explanation
Description Check βœ… Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check βœ… Passed The title clearly matches the main change: upgrading Maven compiler source and target from Java 1.7 to 1.8.
Docstring Coverage βœ… Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check βœ… Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check βœ… Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
πŸ§ͺ Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❀️ Share

Comment @coderabbitai help to get the list of available commands.

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