Closed nevillelyh closed 6 years ago
Can one of the admins verify this patch?
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!
) and we'll verify. Thanks.
I signed it!
CLAs look good, thanks!
Just realized that this fixes the same thing as #74 but I believe this is a cleaner fix.
Is this still maintained? @kchodorow please take a look maybe?
Can one of the admins verify this patch?
@aehlig @dslomov
I'll merge this if nobody objects in the next two days.
@ulfjack did you mean "in the next two = 2 or 20 days" ? :smiley_cat:
Apparently I can't merge it. :-/
Repository
checksgetId
for equality http://grepcode.com/file/repo1.maven.org/maven2/org.apache.maven/maven-model/3.3.3/org/apache/maven/model/RepositoryBase.java#RepositoryBase.equals%28java.lang.Object%29Without this, the
toSet
always end up with 1 repo https://github.com/bazelbuild/migration-tooling/pull/76/files#diff-4518b099acd871fca1a03fff14b3df1bL88