Jonathon Belotti

An aspect [can require a toolchain to be available during execution](https://docs.bazel.build/versions/master/skylark/lib/globals.html#aspect), so this aspect should depend on the Python toolchain. A decent example of how this is done is [in...

Generated Protobuf library code is not compatible with typeshed's protobuf type stubs

1

Not too confident this will be easily fixed on the `typeshed` side -> https://github.com/python/typeshed/issues/2521

Remove need for --output_groups=+mypy

1

Hopefully Aspects are allowed to return `DefaultInfo`, and I can do away with having the `mypy` output group which must be requested explicitly with `--output_groups=+mypy`.

Datadog Dashboards can have non-numeric IDs, but this isn't supported

2

🐛 `ris-aew-hdc` is an example. The Datadog docs now show this type of ID as well: https://docs.datadoghq.com/api/?lang=python#get-a-dashboard ### Actual Behaviour ``` ./dd2tf dashboards --ids ris-aew-hdc invalid argument "ris-aew-hdc" for "--ids"...

**Description** Would appreciate having somewhere (maybe `README`) comparing this project to the [Kubeflow Pipelines Python SDK](https://www.kubeflow.org/docs/pipelines/sdk/sdk-overview/) to say why you would use one or the other.

Update README.md to warn of API restriction that breaks functionality

1

LinkedIn has heavily restricted there API so much of this repo's functionality is now broken. I found out the hard way but this repo should really now have a warning...

Support for tar.gz

1

#### Description I can see for [this line in the source](https://github.com/tubular/rules_pygen/blob/master/src/rules_pygen/rules_generator.py#L82) that you're aware of the problem of handling packages that provide only a `.tar.gz`. I just ran into this...

error: invalid command 'bdist_wheel'

3

Ran this within our Bazel repo and it failed at the following place: https://github.com/tubular/rules_pygen/blob/bdbf28f4bc9fe52010e28fd8d1cc919fdf569dda/src/rules_pygen/rules_generator.py#L364. The error message was saying `Failed to build wheel X` for a number of packages listed...

Filesystem.destroy() is not always called when using BackgroundSession

1

https://github.com/cberner/fuser/pull/154 (**Ensure that Filesystem.destroy() is always called**) made `destroy()` an exactly once method for `Session`, but `BackgroundSession` still relies on the `ll::Operation::Destroy` request to be received from the kernel. We...