Skip to content

Generic Python linter#82

Draft
Pierre Siddall (Pierre-siddall) wants to merge 14 commits into
MetOffice:mainfrom
Pierre-siddall:python-linter
Draft

Generic Python linter#82
Pierre Siddall (Pierre-siddall) wants to merge 14 commits into
MetOffice:mainfrom
Pierre-siddall:python-linter

Conversation

@Pierre-siddall

@Pierre-siddall Pierre Siddall (Pierre-siddall) commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

PR Summary

Code Reviewer:

Similarly to the Fortran linting workflow this PR implements a generic Python linter using ruff and it's associated documentation

This workflow can take in the following input parameters when called by another workflow in a different repository:

Input Parameters

Parameter Description Required Default Type
runner The runner to use for the job No ubuntu-latest string
timeout Maximum time in minutes the job can run No 10 number
ruff-version Ruff linter version No 0.15.0 string
source-path Path to source code directory No . string
fail-on-error Fail workflow on linting errors No true boolean

closes #81

Code Quality Checklist

(Some checks are automatically carried out via the CI pipeline)

  • I have performed a self-review of my own code
  • My code follows the project's style guidelines
  • The modified workflow's README has been updated, if required
  • The changes have been sufficiently tested (please describe)

AI Assistance and Attribution

  • Some of the content of this change has been produced with the assistance
    of Generative AI tool name (e.g., Met Office Github Copilot Enterprise,
    Github Copilot Personal, ChatGPT GPT-4, etc) and I have followed the
    Simulation Systems AI policy
    (including attribution labels)

Code Review

  • The changes are approriate and testing has been sufficient

@Pierre-siddall Pierre Siddall (Pierre-siddall) marked this pull request as draft June 11, 2026 10:16
@Pierre-siddall Pierre Siddall (Pierre-siddall) marked this pull request as ready for review June 11, 2026 12:55
@Pierre-siddall Pierre Siddall (Pierre-siddall) marked this pull request as draft June 11, 2026 12:56
@Pierre-siddall Pierre Siddall (Pierre-siddall) marked this pull request as ready for review June 11, 2026 14:17
@Pierre-siddall Pierre Siddall (Pierre-siddall) marked this pull request as draft June 24, 2026 08:06
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.

Generic Python linting using Ruff

2 participants