Skip to content

refactor log4j-jpa to use junit5#3060

Merged
ppkarwasz merged 1 commit intoapache:2.xfrom
neighbourhoodie:jpa-junit5
Oct 7, 2024
Merged

refactor log4j-jpa to use junit5#3060
ppkarwasz merged 1 commit intoapache:2.xfrom
neighbourhoodie:jpa-junit5

Conversation

@hulkoba
Copy link
Copy Markdown
Contributor

@hulkoba hulkoba commented Oct 7, 2024

Hello 👋

This PR migrates the tests located in log4j-jpa to JUnit5.

Checklist

  • Base your changes on 2.x branch if you are targeting Log4j 2; use main otherwise
  • ./mvnw verify succeeds (if it fails due to code formatting issues reported by Spotless, simply run ./mvnw spotless:apply and retry)
  • Non-trivial changes contain an entry file in the src/changelog/.2.x.x directory
  • Tests for the changes are provided
  • Commits are signed (optional, but highly recommended)

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 7, 2024

Job Requested goals Build Tool Version Build Outcome Build Scan®
build-macos-latest clean install 3.9.8 Build Scan PUBLISHED
build-ubuntu-latest clean install 3.9.8 Build Scan PUBLISHED
build-windows-latest clean install 3.9.8 Build Scan PUBLISHED
Generated by gradle/develocity-actions

@ppkarwasz ppkarwasz merged commit d451aef into apache:2.x Oct 7, 2024
@ppkarwasz
Copy link
Copy Markdown
Contributor

@hulkoba,

Thank You for your contribution. Note that the log4j-jpa artifact was removed from the main branch, so there is no need to port this to main.

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