confluentinc / confluent-docker-utils

Common Python utils for testing Confluent's Docker images
Apache License 2.0
2 stars 3 forks source link

fix: add `MANIFEST.in` file for packaging #85

Closed 0xDEC0DE closed 2 months ago

0xDEC0DE commented 2 months ago

This should allow sdists to install correctly.

Fixes: Issue #84

cla-assistant[bot] commented 2 months ago

CLA assistant check
All committers have signed the CLA.

0xDEC0DE commented 2 months ago

I've signed the CLA, but it doesn't seem to be updating.

No idea how to address this.

cchristous commented 2 months ago

/sem-approve

cchristous commented 2 months ago

/sem-approve

cchristous commented 2 months ago

/sem-approve

cchristous commented 2 months ago

/sem-approve

cchristous commented 2 months ago

I've signed the CLA, but it doesn't seem to be updating.

No idea how to address this.

Hi, thanks for the contribution. Could you attempt to sign the CLA again? I don't see a record of you signing it for this project.

0xDEC0DE commented 2 months ago

Hi, thanks for the contribution. Could you attempt to sign the CLA again? I don't see a record of you signing it for this project.

OK, I have it sorted; it was a personal account vs. work account thing, and apparently the robots don't like that.

I reset the commits to all have the same author and it appears to have worked itself out.

cchristous commented 2 months ago

/sem-approve

cchristous commented 2 months ago

Well, clearly no external contributor has committed to this repo in quite some time. The CI system didn't work at first, which I fixed, and now the tests failed.

I copied the change into https://github.com/confluentinc/confluent-docker-utils/pull/89 to give you the desired functionality. We will fix the tests in parallel