Skip to content

Remove Hamcrest dependency and replace with JUnit assertions#477

Merged
slachiewicz merged 2 commits intocodehaus-plexus:masterfrom
slachiewicz:remove-hamcrest-dependency
Apr 4, 2026
Merged

Remove Hamcrest dependency and replace with JUnit assertions#477
slachiewicz merged 2 commits intocodehaus-plexus:masterfrom
slachiewicz:remove-hamcrest-dependency

Conversation

@slachiewicz
Copy link
Copy Markdown
Member

No description provided.

@slachiewicz slachiewicz force-pushed the remove-hamcrest-dependency branch from d620809 to 008ccc6 Compare April 4, 2026 19:29
Removed Hamcrest dependency from 6 POM files:
- pom.xml (parent POM dependencyManagement)
- plexus-compiler-api/pom.xml
- plexus-compiler-test/pom.xml
- plexus-compilers/plexus-compiler-eclipse/pom.xml
- plexus-compilers/plexus-compiler-javac/pom.xml
- plexus-compilers/plexus-compiler-csharp/pom.xml

Converted all test files from Hamcrest to JUnit assertions:
- plexus-compiler-api: 4 test files
- plexus-compiler-test: 2 test files
- plexus-compiler-eclipse: 3 test files
- plexus-compiler-javac: 2 test files
- plexus-compiler-csharp: 2 test files

All tests pass successfully.
@slachiewicz slachiewicz force-pushed the remove-hamcrest-dependency branch from 008ccc6 to 3aafde2 Compare April 4, 2026 19:32
@slachiewicz slachiewicz merged commit a4e9ef1 into codehaus-plexus:master Apr 4, 2026
29 checks passed
@slachiewicz slachiewicz deleted the remove-hamcrest-dependency branch April 4, 2026 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant