Update version to 1.3.0 #19
Conversation
Required for tagging JUnit 4 tests
This is part of the release process
| * Empty interface, required, with the Category annotation, to label JUnit 4 tests in the following | ||
| * manner: @Category(MaintainedByDiffblue.class) | ||
| */ | ||
| public interface MaintainedByDiffblue {} |
There was a problem hiding this comment.
❔ This is probably something you thought through/tried but I would rather ask - is it enough to have this as an interface or does it need to be a class?
There was a problem hiding this comment.
It can be either. The original documentation I found had it as an interface so I did that but I've tested it as a class as well
There was a problem hiding this comment.
Ah cool, if the suggestion in docs is to use an interface (or if JUnit itself uses it) then presumably that's better standard 👍
There was a problem hiding this comment.
Any of class, interface, enum or annotation would do, in fact (or even a record in Java 17 or later) - all we need is that we can get a Class object for it.
| @@ -16,7 +16,7 @@ | |||
| <modelVersion>4.0.0</modelVersion> | |||
| <groupId>com.diffblue.cover</groupId> | |||
| <artifactId>cover-annotations</artifactId> | |||
| <version>1.2.0</version> | |||
| <version>1.3.0</version> | |||
There was a problem hiding this comment.
@ThomasPerkins1123 from process point of view, are you happy for this commit to be part of the same PR or should it be a standalone PR?
There was a problem hiding this comment.
Yes I am happy with this on this repo, as long as its always its own commit
|
Oh I just noticed that the PR title could be improved, minor thing, otherwise go for it 👍 |
Adds the
MaintainedByDiffblueinterface. This is an empty interface that is require when using the@Categoryannotation for marking JUnit 4 tests.