Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix versioneer #448

Merged
merged 2 commits into from
Nov 30, 2020
Merged

Fix versioneer #448

merged 2 commits into from
Nov 30, 2020

Conversation

jakirkham
Copy link
Member

Fixes #336

  • Adds missing MANIFEST.in to include versioneer components
  • Always uses versioneer in setup.py

Based on local testing this should fix the issue. Also this matches with versioneer's install instructions.

cc @pentschev @quasiben

Some of the components of versioneer may not have been included during
packaging. To fix this we add a `MANIFEST.in` file to ensure these
pieces are included.
Make sure that we always query versioneer for the version. It already
handles cases where the package does not live in version control (like
being installed or in a tarball).
Copy link
Member

@quasiben quasiben left a comment

Choose a reason for hiding this comment

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

Thanks @jakirkham ! LGTM

Copy link
Member

@pentschev pentschev left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @jakirkham !

@codecov-io
Copy link

codecov-io commented Nov 30, 2020

Codecov Report

Merging #448 (beee114) into branch-0.17 (a27bd93) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           branch-0.17     #448   +/-   ##
============================================
  Coverage        89.86%   89.86%           
============================================
  Files               15       15           
  Lines             1095     1095           
============================================
  Hits               984      984           
  Misses             111      111           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a27bd93...beee114. Read the comment docs.

@pentschev pentschev merged commit d4f0cd5 into rapidsai:branch-0.17 Nov 30, 2020
@jakirkham jakirkham deleted the fix_versioneer branch November 30, 2020 20:12
@jakirkham
Copy link
Member Author

Thanks all! 😄

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.

Odd __version__ reported
4 participants