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

Reconfigure CI and Documentation #692

Merged
merged 3 commits into from
Jan 5, 2024

Conversation

Zeitsperre
Copy link
Collaborator

@Zeitsperre Zeitsperre commented Jan 3, 2024

Overview

This Pull Request adds a linting step to the CI configuration and also adds a ReadTheDocs v2 configuration (needed since 2020).

Additional Information

I've set ReadTheDocs to build Pull Requests, but since this PR is coming from a fork, I don't think that ReadTheDocs's configuration will change until the changes here are merged to master.

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 Jan 3, 2024

Coverage Status

coverage: 84.061%. remained the same
when pulling 758ba18 on Zeitsperre:CI-and-Documentation
into 31bf412 on geopython:main.

@Zeitsperre Zeitsperre merged commit 10dd07a into geopython:main Jan 5, 2024
7 checks passed
@Zeitsperre
Copy link
Collaborator Author

@cehbrecht It looks as though the webhook isn't synchronized properly on ReadTheDocs (I can't see the main branch as an option for builds). Think you could try resyncing the GitHub integration? (I would also need maintainer status on GitHub as well to do this myself.)

It can be found here: https://readthedocs.org/dashboard/pywps/integrations/81509/

@cehbrecht
Copy link
Collaborator

cehbrecht commented Jan 10, 2024

@Zeitsperre I tried to re-sync the github integration ... but it didn't help. RTD might got confused with the branches ... we have now a "main" branch ... not "master". Should we remove the integration and re-add it again? I have given you permissions on GitHub as well.

@Zeitsperre
Copy link
Collaborator Author

@cehbrecht I've sent a bug report through ReadTheDocs so hopefully I'll receive a response. This is very strange behaviour, but I'm sure they can help us out with this.

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