Repository navigation
fix(renovate): formater renovate.json selon Biome — l'unique erreur de lint-and-test - #264
Merged
Merged
Conversation
…nt-and-test Biome verifie TOUS les fichiers du depot, JSON compris, et exigeait le tableau extends sur une ligne. C'etait l'unique ERREUR du job (les 247 autres diagnostics sont des warnings), et elle vivait sur dev : les 3 PR Renovate rouges ne touchaient pas ce fichier, elles heritaient du rouge de la base. Le fichier vient du sweep d'harmonisation du portefeuille, qui l'ecrit multi-ligne. C'est donc notre propre standard qui rendait ce depot rouge.
clippy::useless_borrows_in_formatting, seule erreur de `cargo clippy -- -D warnings`. Elle etait cachee DERRIERE l'erreur Biome : le job s'arretait avant d'y arriver. Aucun changement de comportement — format! deref deja l'argument.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Le constat
Les 3 PR Renovate ouvertes (#261 react-router v8 security, #262 motion, #263 non-major) sont toutes rouges sur
lint-and-test. Aucune des trois ne touche le fichier fautif — elles heritent du rouge dedev.La cause
biome checkverifie tous les fichiers du depot, JSON compris. Sur 540 fichiers il rend 247 warnings et une seule erreur :Seules les erreurs font echouer le job ; les 247 warnings passent. Le tableau
extendstient sur une ligne, Biome exige donc qu'il y soit.Ce que ca dit du portefeuille
renovate.jsonvient du sweep d'harmonisation, qui l'ecrit multi-ligne. C'est donc notre propre standard qui rendait ce depot rouge, et il l'etait avant les PR qu'on accusait.Mesure de l'etendue : sur 234 depots non archives, 4 utilisent Biome — FlowForge, Yendor, og-engine, GitHubSkillsAutomate. Seul FlowForge etait rouge ; Yendor#49 et og-engine#55 viennent d'etre merges verts. Le motif ne se repete donc pas ailleurs aujourd'hui.
renovate_v2.py deployest create-if-absent : il ne reecrira pas ce fichier. Si ce contrat changeait un jour, ce correctif serait annule en silence.Suite
Une fois merge, les 3 PR Renovate seront rebasees (case
rebase-check, jamaisupdate-branch) et devraient repasser vertes.