Andrew Graham-Yooll
Andrew Graham-Yooll
> I guess specifying the supported versions would be a plus. How about on the README? > There's a base requirement file that is empty Yeah its empty except for...
> The requirements file, should mention what versions of the packages you are using. I do think this is a valid point though. And I will leave the issue open...
After writing [this](https://github.com/mercadona/rele/issues/185#issuecomment-700709587), then I dont see why we cant pin core dependencies (google-auth and google-pubsub) and leave unrelated stuff like Django, Flask, etc. unpinned.
No worries, we always appreciate the input! Just looking at some other popular projects out there, it seems like they mostly "soft" pin their core dependencies. - https://github.com/celery/celery/blob/master/requirements/default.txt - https://github.com/encode/django-rest-framework/blob/master/setup.py#L85...
_In theory_ this should work without issue according to this https://github.com/mercadona/rele/blob/master/rele/client.py#L62 Have you tried it already?
Ah right, having thought about this for aa few minutes, this makes perfect sense. The reason we create a subscription is due to when the worker spin ups the first...
@daniel-ruiz this issue can be closed 😉
Sounds like a great idea @antoniobusrod ! To take some inspiration from others, we can look at the way Dramatiq handles this [here](https://github.com/Bogdanp/django_dramatiq/blob/master/django_dramatiq/models.py#L31)
Good source @daniel-ruiz ! They also have fairly simple travis build which does use tox. https://github.com/marshmallow-code/marshmallow/blob/dev/.travis.yml
Just because they have removed it from their repo, here is the travis file https://github.com/marshmallow-code/marshmallow/commit/ead95c2624b66ec9ab2fa9c83b55e60929599c2a Curious that they migrated from travis to azure pipelines. Seems to be the number of...