Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@MaribelleHGomez
Copy link
Contributor

@MaribelleHGomez MaribelleHGomez commented Jul 6, 2021

Modified .travis file for poetry use
Changed jwt auth_header test to account for change in return type

Please make sure to follow the DEV guidelines before asking for review.

New Features

  • Now uses poetry for package management

Breaking Changes

pyJWT encode() now returns string instead of byte array

Bug Fixes

Improvements

Migrated to poetry for package management

Dependency updates

  • pyJWT
  • cryptography

Deployment changes

@github-actions
Copy link

github-actions bot commented Jul 6, 2021

The style in this PR agrees with black. ✔️

This formatting comment was generated automatically by a script in uc-cdis/wool.

- sed -i.bak "s/=get_version()/='$TRAVIS_TAG'/g" setup.py
- cat setup.py
- if [ $(python setup.py --version) == '0.0.0' ]; then travis_terminate 1; fi
- poetry config http-basic.pypi $PYPI_USER $PYPI_PASSWORD
Copy link
Contributor

Choose a reason for hiding this comment

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

PYPI_USER and PYPI_PASSWORD need to be added to the encrypted travis env vars

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks

Copy link
Contributor

@paulineribeyre paulineribeyre left a comment

Choose a reason for hiding this comment

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

lgtm 😄

@MaribelleHGomez MaribelleHGomez merged commit d3c1590 into master Jul 7, 2021
@MaribelleHGomez MaribelleHGomez deleted the chore/update_crypto branch July 7, 2021 21:03
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