Skip to content

Conversation

@nicoonoclaste
Copy link
Collaborator

This addresses the feedback I left Piper on the 1.0 release PR, i.e. the version info is now exposed programmatically without duplicating it across setup.cfg and __init__.py.

This addresses the [feedback] I left Piper on the 1.0 release PR:
i.e. the version info is now exposed programatically without duplicating it.

[feedback]: ppb#23 (comment)
In order for `version = attr: ppb_vector.__version__` to work, setuptools
needs to be able to import `pbb_vector`, so its runtime dependencies are
now setup-time dependencies too.
Copy link
Collaborator

@pathunstrom pathunstrom left a comment

Choose a reason for hiding this comment

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

LGTM

@nicoonoclaste
Copy link
Collaborator Author

Thanks to @duckinator for figuring this out <3

@AstraLuma
Copy link
Member

bors r+

bors bot added a commit that referenced this pull request Jun 26, 2019
176: Expose the version number programatically. r=astronouth7303 a=nbraud

This addresses the [feedback] I left Piper on the 1.0 release PR, _i.e._ the version info is now exposed programmatically without duplicating it across `setup.cfg` and `__init__.py`.

[feedback]: #23 (comment)

Co-authored-by: Nicolas Braud-Santoni <nicolas@braud-santoni.eu>
@bors
Copy link
Contributor

bors bot commented Jun 26, 2019

Build succeeded

  • FreeBSD PYTHON:3.6
  • FreeBSD PYTHON:3.7
  • lint
  • docs
  • Linux python:3.7-slim
  • Linux python:3.6-slim
  • Windows python:3.6-windowsservercore-1809
  • macOS PYTHON:3.7.2
  • macOS PYTHON:3.6.8
  • Windows python:3.7-windowsservercore-1809

@bors bors bot merged commit 3f67c14 into ppb:master Jun 26, 2019
@nicoonoclaste nicoonoclaste deleted the version branch June 26, 2019 20:42
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