search

bazeltools / bazel-deps

Generate bazel dependencies for maven artifacts
MIT License
250 stars 121 forks source link

gradle resolver refactors #376

Closed johnynek closed 8 months ago

johnynek commented 8 months ago

I didn't find any smoking gun errors, and couldn't seem to actually reproduce the error we saw.

From my tests, it seems like the code is working.

This PR mainly does 3 things:

  1. simplifies the code and modernizes some syntax
  2. adds tests for gradle resolvers especially exercising ignoring edges
  3. changes the Writer to emit all (not just the first) error it finds.