Skip to content

CI: Bind documentation job to explicit environment context (#7288) - #7439

Closed
ArthPatel1502 wants to merge 0 commit into
OSGeo:mainfrom
ArthPatel1502:feature/use-environment-secrets-7288
Closed

ArthPatel1502 wants to merge 0 commit into
OSGeo:mainfrom
ArthPatel1502:feature/use-environment-secrets-7288

Conversation

@ArthPatel1502

Copy link
Copy Markdown
Contributor

This PR resolves the issue by adding an explicit environment definition to the documentation workflow, matching the project requirements.

Closes #7288

@github-actions github-actions Bot added CI Continuous integration Python Related code is in Python libraries tests Related to Test Suite labels May 29, 2026
@echoix

echoix commented May 29, 2026

Copy link
Copy Markdown
Member

You have extra files unrelated to the change commited here

@echoix

echoix commented May 29, 2026

Copy link
Copy Markdown
Member

Once cleaned up, then we’ll need someone with admin rights to add the environment in the settings. I can’t do it on my own.

@echoix

echoix commented May 29, 2026

Copy link
Copy Markdown
Member

It might be a good idea to make a list/table in the issue of what secrets need to be in which environments, so they could be set up, and do all the changes in a single, focused PR. For sure there isn’t only the docs one.

@ArthPatel1502
ArthPatel1502 force-pushed the feature/use-environment-secrets-7288 branch from fc90a48 to ec30ce7 Compare June 1, 2026 03:19
@ArthPatel1502

Copy link
Copy Markdown
Contributor Author

Apologies for the noise! I have successfully hard-reset the branch against upstream main to completely remove the unrelated CLI and refactor commits. I've also re-applied the environment parameter to the documentation workflow with correct indentation to fix the check failure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous integration libraries Python Related code is in Python tests Related to Test Suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat] Use secrets defined in an environment

2 participants