-
Notifications
You must be signed in to change notification settings - Fork 71
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
Use junit-bom to set versions of junit dependencies #346
Conversation
This PR adds the following to the generated
Then the versions of the dependencies are left defined by the |
The |
In my opinion it's not, as you overwrite the files setting with your local git-configuration, but we discuss this in #341. @pr I like the idea, will have a look at it! |
I 100% agree and I can remove it from the PR. I needed this fix in order to be able to pass the build locally. Now that it's passed in the CI, I don't need it. But, I assume you are not testing on Windows, only Ubuntu, or you would catch the failure. |
I mean, I agree it is not part of this PR. I am not sure I agree that it's overwritten (this does not seem to be the case). |
The |
@@ -106,6 +106,7 @@ The least we can do is to thank them and list some of their accomplishments here | |||
* [Allon Murienik](https://github.com/mureinik) contributed [the range sources](https://junit-pioneer.org/docs/range-sources/) (#44 / #123) | |||
* [Bradford Hovinen](https://github.com/hovinen) improved the execution of the EnvironmentVariableUtils on different OS (#287 / #288) | |||
* [Daniel Kraus](https://github.com/beatngu13) contributed [the system property extension](https://junit-pioneer.org/docs/system-properties/) (#129 / #133), further improved it as well as the environment variable extension (#180 / #248), and helped with build infrastructure (e.g. #269) | |||
* [David Walluck](https://github.com/dwalluck) introduced JUnit 5 BOM (#343 / #346) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'd like to list your contribution @dwalluck, is that ok with you?
@nicolaiparlog It seemed that |
Kudos, SonarCloud Quality Gate passed! 0 Bugs |
Thanks @dwalluck, I streamlined the configuration a bit and added you to the list of contributors (see ☝️). As soon as David ok'ed his inclusion, I consider this done and ready to be merged. |
I'm not sure who started it, but I like it for the clarity - seeing |
PR checklist
The following checklist shall help the PR's author, the reviewers and maintainers to ensure the quality of this project.
It is based on our contributors guidelines, especially the "writing code" section.
It shall help to check for completion of the listed points.
If a point does not apply to the given PR's changes, the corresponding entry can be simply marked as done.
Documentation (general)
.adoc
file in thedocs
folder, e.g.docs/report-entries.adoc
.adoc
files)Documentation (new extension)
docs/docs-nav.yml
navigation has an entry for the new extensionpackage-info.java
contains information about the new extensionCode
Contributing
README.md
mentions the new contribution (real name optional)I hereby agree to the terms of the JUnit Pioneer Contributor License Agreement.