-
Notifications
You must be signed in to change notification settings - Fork 28
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
Order list of rake installations for the dropdown #16
Conversation
plugins » rake-plugin #4 SUCCESS |
@Override | ||
public int compareTo(RubyInstallation other){ | ||
int last = this.name.compareTo(other.name); | ||
return last == 0 ? this.name.compareTo(other.name) : last; |
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.
Isn't the second call to compareTo()
redundant, especially since name
is immutable? In other words, why couldn't we simply return last
?
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.
good point! I will change that.
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.
done
- This change removes the redundation call to String compareTo - It also adds a test for the compare to function
For some reason the pull request is not picking up my latest commit on my source branch. |
Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests |
plugins » rake-plugin #5 SUCCESS |
plugins » rake-plugin #6 SUCCESS |
Order list of rake installations for the dropdown
No description provided.