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
Handle tags windup 679 #633
Conversation
f59f06f
to
0445e82
Compare
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
f397ea6
to
69f1b55
Compare
Refer to this link for build results (access rights to CI server needed): Failed Tests: 1org.jboss.windup.rules:windup-rulesets: 1Test FAILed. |
Refer to this link for build results (access rights to CI server needed): Failed Tests: 1org.jboss.windup.rules:windup-rulesets: 1Test FAILed. |
69f1b55
to
ae5bf6c
Compare
@@ -178,6 +190,11 @@ public int getEffort() | |||
return links; | |||
} | |||
|
|||
public Set<String> getTags() | |||
{ | |||
return UnmodifiableSet.decorate(tags); |
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.
Why use apache-commons, when you have
java.util.Collections.unmodifiableSet(Set<? extends T>)
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.
That's right, fixed
bf82d0b
to
0cc276a
Compare
2ce34d6
to
7fc2298
Compare
7fc2298
to
82720a5
Compare
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): Failed Tests: 1org.jboss.windup.rules:windup-rulesets: 1Test FAILed. |
For now, it is storing the tags in @SetInProperties.
What's new: