Skip to content

Conversation

@itsyarkee
Copy link
Contributor

changing setup.py to import the version from kafka

wizzat added a commit that referenced this pull request Aug 3, 2014
@wizzat wizzat merged commit 7bfdafa into dpkp:master Aug 3, 2014
@dpkp
Copy link
Owner

dpkp commented Aug 3, 2014

actually this is the wrong way to do this. see "warning" in answer here: http://stackoverflow.com/questions/2058802/how-can-i-get-the-version-defined-in-setup-py-setuptools-in-my-package

@wizzat
Copy link
Collaborator

wizzat commented Aug 3, 2014

Interesting. I've seen a lot of packages built the way this PR is done. I kinda like the VERSION approach though.

orange-kao pushed a commit to orange-kao/kafka-python that referenced this pull request Jul 23, 2024
* stop pylint complaint for uncovered conditional flow

* add todo to revisit

* formatting makes me happy :)

* Fix errors raised by new version of Pylint so tests pass again
orange-kao added a commit to orange-kao/kafka-python that referenced this pull request Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants