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

Feature/deploy #20

Open
wants to merge 45 commits into
base: master
Choose a base branch
from
Open

Feature/deploy #20

wants to merge 45 commits into from

Conversation

alshapton
Copy link

This PR addresses Issue #15 .

It:

  • Checks the code using SonarCloud
  • Uses flake8 to check for Python syntax errors
  • Uses flake8 to scan the code (in addition to SonarCloud)
  • Runs the built-in tests with Pytest
  • Runs accessibility checks with pa11y
  • Publishes to PyPI (if on a release branch)
  • Publishes rudimentary documentation to READTHEDOCS.org (if on a release branch)

This build pipeline runs on a matrix of MacOS/Ubuntu with Python versions: 3.7, 3.8, 3.9, 3.10 and pypy-3.8

It also introduces the GitFlow branching strategy (to be fully implemented within the main repository by @mrwilson, however the travis build is deprecated and will be removed in a future PR)

There is a new CONTRIBUTING.md file
Restructured directories slightly

…ectly)

* Added tooltips to icons and tested all with ChromeVox Screenreader
* Changed <i..... class to <em>.....</em>
'
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