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

packaging primer #164

Merged
merged 4 commits into from
Apr 11, 2020
Merged

packaging primer #164

merged 4 commits into from
Apr 11, 2020

Conversation

daimesava
Copy link
Contributor

makes pymaker available as a python package, so it can be installed via pip or included in a project's requirements file. e.g:

pip install git+https://github.com/makerdao/pymaker

@daimesava daimesava requested review from grandizzy and EdNoepel April 9, 2020 15:12
@codecov-io
Copy link

codecov-io commented Apr 9, 2020

Codecov Report

Merging #164 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #164   +/-   ##
=======================================
  Coverage   89.26%   89.26%           
=======================================
  Files          30       30           
  Lines        4303     4303           
=======================================
  Hits         3841     3841           
  Misses        462      462           

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 69c7b6d...4d1a910. Read the comment docs.

@grandizzy grandizzy requested a review from MikeHathaway April 9, 2020 15:32
Copy link
Contributor

@grandizzy grandizzy left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@EdNoepel EdNoepel left a comment

Choose a reason for hiding this comment

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

Are we going to publish this library to PyPI? Does the foundation have a login for this?

setup.py Outdated
# It ensures open() defaults to text mode with universal newlines,
# and accepts an argument to specify the text encoding
# Python 3 only projects can skip this import
from io import open
Copy link
Contributor

Choose a reason for hiding this comment

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

We can skip this, as we don't support Python 2.7.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I missed it. Removed

setup.py Outdated
# For a discussion on single-sourcing the version across setup.py and the
# project code, see
# https://packaging.python.org/en/latest/single_source_version.html
version='1.0.4', # Required
Copy link
Contributor

Choose a reason for hiding this comment

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

Curious how version was determined, and how we should update it going forward.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pulled it from cf515c9 but I'm not sure of coupling pymaker versioning to MCD releases is future-proof.

Copy link
Contributor

Choose a reason for hiding this comment

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

See discussion here: #75

@daimesava
Copy link
Contributor Author

Are we going to publish this library to PyPI? Does the foundation have a login for this?

For now this doesn't publish anywhere, but happy to take this further and set up pypi for this and other pkgs lilke the auction-keeper.

@EdNoepel
Copy link
Contributor

EdNoepel commented Apr 9, 2020

I think there's value adding this to PyPI. I'd just like to understand how we'll push new releases. Is that something techops would handle, or will other contributors within the foundation be able to push? Maybe we should start the version at 1.0.0?

Copy link
Contributor

@EdNoepel EdNoepel left a comment

Choose a reason for hiding this comment

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

Please let me know what our process will be for publishing new releases to PyPI. Happy to discuss with you at your convenience.

@EdNoepel EdNoepel merged commit 163a03a into makerdao:master Apr 11, 2020
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.

4 participants