CI improvements#703
Open
Zeitsperre wants to merge 4 commits into
Open
Conversation
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
fmigneault
approved these changes
Jun 29, 2026
fmigneault
left a comment
Member
There was a problem hiding this comment.
PR looks fine for the CI part.
@Zeitsperre
I do not have admin access to grant GeoPython access.
cehbrecht
approved these changes
Jun 30, 2026
Collaborator
|
@Zeitsperre I have not seen a request ... maybe it goes to @tomkralidis ?
|
Member
|
I'll take a look at the ReadTheDocs updates and how they affect other projects. |
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Collaborator
Author
|
@tomkralidis Thanks! Please let me know when you've had a look at things. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Overview
ThiS PR adds a few small changes to ensure that commits are used for version control instead of tags.
It also adds a few updates to the docs-building step, relying solely on
toxandmakefor CI purposes (easier to configure).Finally, recent changes to
macos-latestrequire that non-standard "taps" be approved. This affects coveralls running on macOS insofar as an additional approval step is now required. Hoepfully the Coveralls devs will adjust this soon.Additional Information
ReadTheDocs is asking that project migrate from the manual OAuth hook method to the new ReadTheDocs App. Sometime last week, manually-configured hooks stopped working globally, so ReadTheDocs needs to be installed in Geopython, at the very least on this project if we want to continue building the docs on PRs and on changes to
main.I've sent a request in that needs to be approved by someone with admin access at geopython. @fmigneault or @cehbrecht, could either of you approve that? I can finish the migration on RTD afterwards.
Contribution Agreement
(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)