Skip to content

Fix failing dependency updates workflow - #2067

Merged
nishtha-kalra merged 6 commits into
mainfrom
nk/fix-yml-file
Sep 23, 2026
Merged

nishtha-kalra merged 6 commits into
mainfrom
nk/fix-yml-file

Conversation

@nishtha-kalra

Copy link
Copy Markdown
Contributor

Configure the setup action to install Python 3.11 before running the dependency update command. The justfile defaults to python3.11, so the workflow failed with "python3.11: command not found" before it could update any dependencies.

Also align the workflow with reports by using the latest Ubuntu runner and limiting the GitHub App token to contents and pull-request write permissions.

Configure the setup action to install Python 3.11 before running the
dependency update command. The justfile defaults to python3.11, so the
workflow [failed](https://github.com/opensafely/documentation/actions/runs/35734406042/job/107089817671)
with "python3.11: command not found" before it could update any dependencies.

Also align the workflow with [reports](https://github.com/opensafely-core/reports/blob/main/.github/workflows/update-dependencies.yml)
by using the latest Ubuntu runner and limiting the GitHub App token to
contents and pull-request write permissions.
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

Deploying opensafely-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 61bda53
Status: ✅  Deploy successful!
Preview URL: https://0ac43ee2.opensafely-docs.pages.dev
Branch Preview URL: https://nk-fix-yml-file.opensafely-docs.pages.dev

View logs

Warning: Input 'app-id' has been deprecated with message: Use
'client-id' instead.
To prevent YAML parsing issues before they become a problem
GitHub Action runner reports: Inputs 'owner' and 'repositories' are
not set. Creating token for this repository
(opensafely/documentation).

@nishtha-kalra nishtha-kalra left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Reviewed @tomodwyer's changes and they look good. Thanks for improving this PR.

@nishtha-kalra
nishtha-kalra merged commit 82cf0af into main Sep 23, 2026
4 checks passed
@nishtha-kalra
nishtha-kalra deleted the nk/fix-yml-file branch September 23, 2026 11:52
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.

2 participants