Project

General

Profile

Development #77685

pre-commit-debian : toujours wrapper

Added by Frédéric Péters 11 months ago. Updated 11 months ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
16 May 2023
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No
Club:
No

Description

La version 2.23.0 de devscripts (qui contient wrap-and-sort, qu'utilise pre-commit-debian) a introduit ce changement :

+ Always apply a trailing comma when run with -t also when the field
is not being wrapped. Closes: #989785; MR: !267

Ça amène une différence de comportement entre l'exécution côté jenkins (avec une version qui ne fait pas ça) et l'exécution en local.

Il n'y a pas de flag qui assurerait la compatibilité avec l'ancien comportement, on pourrait attendre la montée de version côté jenkins mais ça signifie continuer avec ce bug encore longtemps. On pourrait plutôt choisir de toujours wrapper, ce qui amènera un résultat identique des deux côtés (mais un changement par rapport à l'existant).

History

#1

Updated by Robot Gitea 11 months ago

  • Status changed from Nouveau to Solution proposée
  • Assignee set to Frédéric Péters

Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :

#2

Updated by Robot Gitea 11 months ago

  • Status changed from Solution proposée to Solution validée

Thomas NOËL (tnoel) a approuvé une pull request sur Gitea concernant cette demande :

#3

Updated by Robot Gitea 11 months ago

  • Status changed from Solution validée to Résolu (à déployer)

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#4

Updated by Frédéric Péters 11 months ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF