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

Modernize CI using Tox, add a "[dev]" recipe, re-enable coveralls #643

Merged
merged 11 commits into from
Dec 22, 2021

Conversation

Zeitsperre
Copy link
Collaborator

Overview

This PR does a bit of cleanup to the package's CI and current tests.

  • Updates the tox.ini to reflect the current state of the package.
  • Adds a [dev] recipe to the setup.py so that developers can install pywps for development with pip install -e (pywps/.)[dev].
  • Removes a few warning raised from the test ensemble by specifying that raw strings are being used with regex.
  • Update the GitHub CI workflow to use tox, re-enabling the Coveralls check (the last successful run was uploaded in July 2020).
  • Updates the License to include $CURRENT_YEAR

Related Issue / Discussion

Additional Information

Contribution Agreement

(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

  • I'd like to contribute [feature X|bugfix Y|docs|something else] to PyWPS. I confirm that my contributions to PyWPS will be compatible with the PyWPS license guidelines at the time of contribution.
  • I have already previously agreed to the PyWPS Contributions and Licensing Guidelines

@coveralls
Copy link

coveralls commented Dec 21, 2021

Coverage Status

Coverage increased (+81.1%) to 81.137% when pulling 8f2fcca on tox_modernisation into c6a0d7e on main.

setup.py Outdated Show resolved Hide resolved
@Zeitsperre Zeitsperre merged commit 7f228ff into main Dec 22, 2021
@Zeitsperre Zeitsperre deleted the tox_modernisation branch December 22, 2021 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants