Skip to content

ci: add publiccode.yml validation workflow - #444

Open
publiccode-pr-bot wants to merge 1 commit into
codegouvfr:mainfrom
publiccode-pr-bot:chore/add-publiccode-parser-action
Open

ci: add publiccode.yml validation workflow#444
publiccode-pr-bot wants to merge 1 commit into
codegouvfr:mainfrom
publiccode-pr-bot:chore/add-publiccode-parser-action

Conversation

@publiccode-pr-bot

Copy link
Copy Markdown

This PR adds a GitHub Actions workflow that validates publiccode.yml on every pull request using italia/publiccode-parser-action@v1.

No behavior change to the code, just CI.

@revolunet

Copy link
Copy Markdown
Collaborator

Thanks but can you pin the action with a fixed SHA ?

@kevbarns

Copy link
Copy Markdown
Collaborator

@revolunet @garronej Cette PR est devenue sans objet et peut être fermée.

La demande de @revolunet du 20/05 (« can you pin the action with a fixed SHA ? ») a été traitée autrement : .github/workflows/publiccodeyml-check.yml a été ajouté sur main par #488 (commit 3590215). Ce workflow fait exactement le même travail, en mieux :

  • italia/publiccode-parser-action épinglé sur 56e1200cba853b1efa73ee871600284d0705ab4d, et actions/checkout sur 11bd719 — ce qui était précisément la demande ;
  • permissions: {} au niveau du workflow et contents: read sur le job, au lieu des permissions par défaut ;
  • déclencheurs restreints via paths à publiccode.yml et au workflow lui-même, au lieu de on: [pull_request, push] sur tout.

Merger #444 en l'état ajouterait un second workflow redondant, sur tags flottants et sur tous les push. Sauf objection, je propose de la fermer en renvoyant vers #488.

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.

3 participants