Skip to content

ci: gère les PR de fork dans le formatage Ruff#26

Merged
CyrilJl merged 1 commit into
mainfrom
fix/fork-pr-ruff-format
Jul 9, 2026
Merged

ci: gère les PR de fork dans le formatage Ruff#26
CyrilJl merged 1 commit into
mainfrom
fix/fork-pr-ruff-format

Conversation

@CyrilJl

@CyrilJl CyrilJl commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • Checkout le dépôt source de la PR au lieu de supposer que la branche existe dans le dépôt principal.
  • Garde l'auto-format + push pour les PR internes.
  • Passe les PR de fork en vérification seule, car le token est en lecture seule et les secrets ne sont pas disponibles.

Validation

  • rtk proxy ruby -e 'require "yaml"; YAML.load_file(".github/workflows/ruff-format-pr.yml"); puts "yaml ok"'
  • rtk proxy git diff --check

Le workflow checkout maintenant le dépôt source de la PR au lieu de supposer que la branche existe dans le dépôt principal.

Les PR internes gardent l'auto-format avec push. Les PR venant d'un fork passent en mode vérification seule, car le token est en lecture seule et les secrets ne sont pas disponibles.
@CyrilJl CyrilJl self-assigned this Jul 9, 2026
@CyrilJl
CyrilJl merged commit d5d0385 into main Jul 9, 2026
4 checks passed
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