Skip to content

chore(dockerfile): filtrer le lint hadolint par paths [FTTECH-10638]#154

Open
florentsorel wants to merge 1 commit into
masterfrom
chore/dockerfile-lint-paths-filter
Open

chore(dockerfile): filtrer le lint hadolint par paths [FTTECH-10638]#154
florentsorel wants to merge 1 commit into
masterfrom
chore/dockerfile-lint-paths-filter

Conversation

@florentsorel

Copy link
Copy Markdown
Contributor

What Does It Do?

Ajoute un filtre paths sur les triggers pull_request et push du workflow Dockerfile (lint hadolint) distribué dans sample/workflows/dockerfile.yml :

paths:
  - "**/Dockerfile*"
  - ".github/workflows/dockerfile.yml"

Le job hadolint ne se déclenche plus sur les PR/push qui ne modifient aucun Dockerfile.

Context

Le snippet était déclenché sur tous les pull_request/push, donc le lint tournait à vide sur des PR sans rapport (constaté sur une PR docs-only de PanoramAI : panoramai#1515). matchFiles: "**/Dockerfile*" côté policy ne gère que l'installation du workflow, pas son déclenchement — le filtre paths doit vivre dans le snippet.

Le trigger merge_group reste non filtré : les required checks évalués en merge queue continuent de tourner (un required check filtré par paths resterait pending et bloquerait le merge). Même pattern que sample/workflows/mobsuccess.yml.

Propagation : une fois mergé, le bot policy re-synchronise le bloc DO NOT EDIT de dockerfile.yml dans chaque repo.

Test Plan

  • Vérifier qu'une PR ne touchant aucun Dockerfile ne déclenche plus le job « Dockerfile »
  • Vérifier qu'une PR modifiant un Dockerfile (à n'importe quel niveau) déclenche bien le lint
  • Vérifier que le job tourne toujours en merge queue (merge_group)

Closes FTTECH-10638

Ajoute un filtre `paths` (**/Dockerfile* + le workflow lui-même) sur les
triggers `pull_request` et `push` du workflow Dockerfile distribué. Le job
hadolint ne se déclenche plus sur les PR/push qui ne touchent aucun
Dockerfile (bruit observé sur des PR docs-only).

`merge_group` reste non filtré : les required checks en merge queue
continuent de tourner, pas de PR bloquée.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 30, 2026 07:05
@github-actions github-actions Bot added the chore Chore feature label Jun 30, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Ce PR met à jour le workflow d’exemple Dockerfile (hadolint) afin d’éviter des exécutions inutiles sur des PR/push ne modifiant aucun Dockerfile, tout en conservant l’exécution en merge queue (merge_group) pour les required checks.

Changes:

  • Ajout d’un filtre paths sur le trigger pull_request pour ne lancer hadolint que si un Dockerfile* (ou le workflow) change.
  • Ajout d’un filtre paths sur le trigger push (branches master et prod) avec la même logique.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

Labels

chore Chore feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants