-
Notifications
You must be signed in to change notification settings - Fork 19
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
Jarviz Updates #4
Jarviz Updates #4
Conversation
* @return true if version is LATEST or RELEASE, false otherwise. | ||
*/ | ||
@Value.Default | ||
default boolean isVersionLatestOrRelease() { |
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.
Since this contain 2, we can keep it as a List, else we can make a it a Set.
@@ -28,6 +28,8 @@ | |||
@JsonInclude(value = JsonInclude.Include.NON_EMPTY) | |||
public interface CouplingRecord { | |||
|
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.
Can you add a Java doc pls?
@@ -73,6 +73,52 @@ public void testToFileName() { | |||
.isEqualTo("foo-bar-2.0.1-SNAPSHOT.jar"); | |||
} | |||
|
|||
@Test |
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.
+1
@@ -59,4 +60,19 @@ public void testDiscoverArtifact_Fail() throws ArtifactNotFoundException { | |||
artifactDiscoveryService.discoverArtifact(artifact); | |||
} | |||
|
|||
@Test |
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.
For this test to be correct shouldn't we remove "throws ArtifactNotFoundException" from the signature?
💯 |
📝 Description
continueOnMavenError
flag inconfig.json
. When set totrue
,Jarvis
will continue processing remaining applications, fromartifact.json
, after it encounters maven errors. Default value isfalse
.RELEASE
andLATEST
as dependency versions inartifact.json
.appSetName
is now written to outputjsonl
file.🔗 Related Issues