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

Add support for Windows, Python 3 and PyPi deployment #4

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

douglaslassance
Copy link

@douglaslassance douglaslassance commented Dec 2, 2021

Very cool project conceptually.
This update carries the following improvements:

  • Formatted code with Black.
  • Ported code for Python >=3.7 (dropped compatibility with Python 2.x).
  • Added GitHub workflows for integration by running tests on pull requests.
  • Added GitHub workflows for deployment by generating docs and publishing on PyPi upon tagging.
  • Added support for Windows systems.

This merge request is designed to be rebased and merged.

Actions will have to be allowed for this repository in order to run.
The following secrets will have to be defined on this project for deployment workflow to function:

  • PYPI_USERNAME
  • PYPI_PASSWORD

Set to draft because 3 tests are still failing.
All changes are up for discussion and improvement of course.

@douglaslassance douglaslassance changed the title ✨ Update for Windows 10, Python 3.7, and PyPi ✨ Update for Windows 10, Python >=3.7, and PyPi Dec 2, 2021
@douglaslassance douglaslassance force-pushed the py37-update branch 6 times, most recently from 161d403 to 60d09e4 Compare December 2, 2021 03:11
@douglaslassance douglaslassance changed the title ✨ Update for Windows 10, Python >=3.7, and PyPi Add support for Windows, Python 3 and PyPi deployment Dec 2, 2021
@douglaslassance douglaslassance force-pushed the py37-update branch 2 times, most recently from 7ac5a01 to d582b39 Compare December 2, 2021 18:57
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.

1 participant