Allow multiple types of metadata files by aronasorman · Pull Request #16 · bazel-contrib/rules_python

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like a good idea. In the unlikely even that a wheel has both, can you structure this to look for pydist.json first, and only look for metadata.json as a fallback?

Also, there's an open PR for Windows compatibility that affects these lines, I think you'll want to change line 70 to something like:

    with whl.open(self._dist_info() + '/' + metadata_file) as f: