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.

Sign up for GitHub

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

Jump to bottom

Reconfigure CI and Documentation #692

Merged
merged 3 commits into from
Jan 5, 2024
Merged

Reconfigure CI and Documentation #692

merged 3 commits into from
Jan 5, 2024

Conversation

Zeitsperre
Copy link
Collaborator

@Zeitsperre Zeitsperre commented Jan 3, 2024 •
edited
Loading

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

@Zeitsperre Zeitsperre self-assigned this Jan 3, 2024
@coveralls
Copy link

coveralls commented Jan 3, 2024 •
edited
Loading

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 •
edited
Loading

@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
Reviewers

@cehbrecht cehbrecht cehbrecht approved these changes

Assignees

@Zeitsperre Zeitsperre

Labels
documentation enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Zeitsperre @coveralls @cehbrecht