Skip to content

Add actionlint configuration and CI workflow - #2397

Closed
chernojagne wants to merge 3 commits into
geopython:masterfrom
autar-pr:autar/setup-actionlint
Closed

Add actionlint configuration and CI workflow#2397
chernojagne wants to merge 3 commits into
geopython:masterfrom
autar-pr:autar/setup-actionlint

Conversation

@chernojagne

Copy link
Copy Markdown

Overview

Adds actionlint configuration and a dedicated
CI workflow to lint the project's GitHub Actions workflow files.

Two files are added:

  • .github/actionlint.yaml — actionlint config with ShellCheck integration enabled.
    Common false-positive ShellCheck rules for GitHub Actions (${{ }} interpolation
    patterns: SC2086, SC2129, SC2155, SC1091) are suppressed to avoid noise from
    legitimate GHA idioms.
  • .github/workflows/actionlint.yml — CI workflow triggered on changes to
    .github/workflows/** and the actionlint config itself. Uses actionlint's own
    upstream download script (scripts/download-actionlint.bash), so the version
    stays current without pinning.

pygeoapi has 6 workflow files covering matrix builds, multi-platform Docker pushes,
and workflow_run triggers — all common sources of hard-to-spot YAML/shell mistakes
that actionlint catches statically.

Related Issue / discussion

No prior issue — this is a straightforward tooling addition with no behaviour change
to the project itself.

Additional information

This change was generated by Autar and reviewed by a human before
submission.

Checklist

  • Dependency policy (RFC2): This PR adds no new Python, JavaScript, or system
    dependencies. actionlint is downloaded at CI time via its own release script and is
    not added to any project requirements file. RFC2 requirements are met.
  • Updates to public demo: No changes to pygeoapi behaviour or configuration —
    the demo server is unaffected.
  • Contributions and licensing: Submitted under the same MIT licence as the
    pygeoapi project, per GitHub's inbound=outbound terms.

Cherno Jagne added 3 commits August 7, 2026 07:19
Sets up actionlint to lint GitHub Actions workflow files on push/PR.
Adds .github/actionlint.yaml config with ShellCheck integration and
.github/workflows/actionlint.yml dedicated CI workflow triggered on
workflow file changes.
The paths: key is not a valid actionlint config option; only
self-hosted-runner, ignore, and config-variables are supported at the
top level. The nested ignore patterns were silently discarded, so
pre-existing SC2006/SC2046 errors in main.yml triggered CI failures.
@chernojagne

Copy link
Copy Markdown
Author

Closing this — we're revising how we approach these contributions and would rather withdraw it than leave it open unattended. Apologies for the noise, and thanks.

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.

1 participant