chore(ci): drop the French comments from the workflows - #248
Conversation
There was a problem hiding this comment.
Comment-only deletion, YAML unchanged and still valid. The pull_request_target trigger, the zizmor ignore and the permissions block all survive intact, so behaviour is identical.
Nit: the first deleted block was the written justification for # zizmor: ignore[dangerous-triggers] — that the called workflow never checks out PR code and only reads the title from the event payload. The suppression now stands with no stated reason, and the next person auditing this file has to re-derive the safety argument. The org rule is what it is and this is a deliberate deletion rather than translation, so I'm not blocking on it, but that particular argument is worth parking somewhere durable (FerrLabs/.github#316, or the reusable workflow's own docs) rather than losing it.
Removes the French comment blocks from this repository's GitHub Actions workflows: 1 file(s), 8 lines.
Deletion rather than translation, per the decision on the tracking issue. The org rule is that everything is written in English, and the code-comment rule is that comments do not belong in the first place, so the two point the same way here.
Detection was calibrated rather than eyeballed. A block counts as French on one elision (
d',l',qu') or two French function words. Accents were deliberately demoted to corroborating evidence only, because several files carry mojibake (UTF-8 decoded as Latin-1) that puts accented characters inside perfectly English comments; scoring on accents alone flagged an entirely English block in UI. Inline trailing comments such as# zizmor: ignore[...]are untouched, since only whole-line blocks are considered.Ref FerrLabs/.github#316