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

Deploy to PyPI #27

Merged
merged 10 commits into from
Feb 18, 2021
Merged

Deploy to PyPI #27

merged 10 commits into from
Feb 18, 2021

Conversation

glow-mdsol
Copy link
Member

Added settings for deployment to PyPI (env vars in travis)
Removed PyPy (building cryptography from source requires rust)
Added Python 3.9

Copy link
Contributor

@ykitamura-mdsol ykitamura-mdsol left a comment

Choose a reason for hiding this comment

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

can you update the version and CHANGELOG?

To update the version, we need to change two files:

__version__ = "1.2.0"

version = "1.2.0"

.travis.yml Outdated
@@ -36,10 +36,13 @@ jobs:
before_deploy:
- poetry config repositories.mdsol https://mdsol.jfrog.io/mdsol/api/pypi/pypi-local
Copy link
Contributor

Choose a reason for hiding this comment

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

@jfeltesse-mdsol Do we still need to publish to Artifactory? I think publishing just to PyPI is enough.

Copy link
Contributor

Choose a reason for hiding this comment

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

agreed, PyPI is enough.

@@ -36,10 +36,13 @@ jobs:
before_deploy:
- poetry config repositories.mdsol https://mdsol.jfrog.io/mdsol/api/pypi/pypi-local
- poetry config http-basic.mdsol $ARTIFACTORY_USERNAME $ARTIFACTORY_PASSWORD # Stored as Travis CI Environment Vars
- poetry config pypi-token.pypi $POETRY_PYPI_TOKEN_PYPI # this may be unnecessary
Copy link
Contributor

Choose a reason for hiding this comment

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

looks like there is an issue with Environment variable POETRY_PYPI_TOKEN_<NAME> so I think this is necessary for now.
python-poetry/poetry#2210

.travis.yml Outdated
Comment on lines 43 to 45
script:
poetry publish -r mdsol
poetry publish
Copy link
Contributor

Choose a reason for hiding this comment

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

- is missing:

Suggested change
script:
poetry publish -r mdsol
poetry publish
script:
- poetry publish -r mdsol
- poetry publish

but if we don't need to publish to Artifactory, it will be just:

script: poetry publish

Copy link
Contributor

@jfeltesse-mdsol jfeltesse-mdsol Feb 11, 2021

Choose a reason for hiding this comment

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

why do we need the script provider to begin with? travis supports PyPI: https://docs.travis-ci.com/user/deployment-v2/providers/pypi/

Copy link
Contributor

Choose a reason for hiding this comment

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

because we are using poetry python-poetry/poetry#669

@ykitamura-mdsol
Copy link
Contributor

fyi @jcarres-mdsol @cmcinnes-mdsol

Copy link
Contributor

@ykitamura-mdsol ykitamura-mdsol left a comment

Choose a reason for hiding this comment

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

looks good to me 👍

# needed for Python < 3.8
import importlib_metadata

__version__ = importlib_metadata.version(__name__)
Copy link
Contributor

Choose a reason for hiding this comment

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

nice! 👍

@jfeltesse-mdsol jfeltesse-mdsol changed the title Feature/deploy to pypi Deploy to PyPI Feb 18, 2021
@jfeltesse-mdsol jfeltesse-mdsol merged commit 9dbc1bd into develop Feb 18, 2021
@jfeltesse-mdsol jfeltesse-mdsol deleted the feature/deploy-to-pypi branch February 18, 2021 03:12
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