Skip to content

CI improvements#703

Open
Zeitsperre wants to merge 4 commits into
mainfrom
ci-improvements
Open

CI improvements#703
Zeitsperre wants to merge 4 commits into
mainfrom
ci-improvements

Conversation

@Zeitsperre

Copy link
Copy Markdown
Collaborator

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 tox and make for CI purposes (easier to configure).

Finally, recent changes to macos-latest require 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)

  • 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

Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
@Zeitsperre Zeitsperre self-assigned this Jun 29, 2026
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
@coveralls

coveralls commented Jun 29, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 80.551%. first build — ci-improvements into main

@fmigneault fmigneault left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR looks fine for the CI part.

@Zeitsperre
I do not have admin access to grant GeoPython access.

@cehbrecht

Copy link
Copy Markdown
Collaborator

@Zeitsperre I have not seen a request ... maybe it goes to @tomkralidis ?

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.

@tomkralidis

Copy link
Copy Markdown
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>
@Zeitsperre

Copy link
Copy Markdown
Collaborator Author

@tomkralidis Thanks! Please let me know when you've had a look at things.

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.

5 participants