Project

General

Profile

Développement #83824

passer à pylint 3.X

Added by Valentin Deniaud about 1 year ago. Updated 3 months ago.

Status:
Nouveau
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
22 November 2023
Due date:
% Done:

0%

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

Description

Il y a eu #81905 pour poser la limite de version en raison d'un retard de pylint-django, désormais résolu.

À priori il faut juste revert les patches partout, puis éventuellement corriger les nouvelles erreurs.

(mais perso jamais compris pourquoi on vivait avec pylint sans limite de version, ce qui casse régulièrement nos builds sans prévenir quand un nouveau warning est introduit)


Related issues

Related to Chrono - Développement #82169: Passer à pylint 3.0Fermé09 October 2023

Actions
Related to Authentic 2 - Développement #84019: tox.ini: retirer la constrainte pylint<3Fermé28 November 2023

Actions
Related to Chrono - Développement #95719: Prendre en charge pylint 3.3.0Fermé20 September 2024

Actions

History

#1

Updated by Valentin Deniaud about 1 year ago

#2

Updated by Frédéric Péters about 1 year ago

(mais perso jamais compris pourquoi on vivait avec pylint sans limite de version, ce qui casse régulièrement nos builds sans prévenir quand un nouveau warning est introduit)

Perso, 1/ je crains que si on mettait une limite de version on ne passerait jamais sur les nouvelles versions (on vit ça à d'autres endroits, genre #82024); 2/ ça évite un moment de grand saut, qui pourrait amener à devoir corriger plein de choses.

#3

Updated by Paul Marillonnet about 1 year ago

Juste par curiosité, est-ce qu’à un moment il y a eu le débat de, quitte à tout chambouler, une montée de version majeure de pylint VS une migration vers ruff ?

#4

Updated by Frédéric Péters about 1 year ago

À ma connaissance on n'a jamais discuté de ruff (qui reste très jeune).

#5

Updated by Valentin Deniaud about 1 year ago

Frédéric Péters a écrit :

Perso, 1/ je crains que si on mettait une limite de version on ne passerait jamais sur les nouvelles versions (on vit ça à d'autres endroits, genre #82024);

J'aurais plutôt fait une analogie avec les pre-commit hooks qu'avec une dépendance du code. Ils sont aussi dans une version fixée et on passe bien les mettre à jour de temps en temps.

2/ ça évite un moment de grand saut, qui pourrait amener à devoir corriger plein de choses.

Entre un grand saut à tête reposée et des mini sauts gérés dans une relative urgence...

#6

Updated by Valentin Deniaud about 1 year ago

Paul Marillonnet a écrit :

quitte à tout chambouler, une montée de version majeure de pylint

Elle n'a de majeur que le nom, aucun chamboulement à prévoir (0 modif nécessaire au code de chrono, en tout cas).

#7

Updated by Frédéric Péters about 1 year ago

Ils sont aussi dans une version fixée

Mais ça n'y est pas notre choix. ("pre-commit configuration aims to give a repeatable and fast experience and therefore intentionally doesn't provide facilities for "unpinned latest version" for hook repositories." -- https://pre-commit.com/#using-the-latest-version-for-a-repository)

#8

Updated by Valentin Deniaud about 1 year ago

Dac, c'est bien que les raisons de la situation actuelle soient explicitées, restons comme ça.

#9

Updated by Paul Marillonnet about 1 year ago

Valentin Deniaud a écrit :

Paul Marillonnet a écrit :

quitte à tout chambouler, une montée de version majeure de pylint

Elle n'a de majeur que le nom, aucun chamboulement à prévoir (0 modif nécessaire au code de chrono, en tout cas).

Ok top.

#10

Updated by Valentin Deniaud about 1 year ago

#11

Updated by Emmanuel Cazenave 3 months ago

#12

Updated by Emmanuel Cazenave 3 months ago

  • Subject changed from passer à pylint 3.0 to passer à pylint 3.X

On en est à 3.3 maintenant.

Also available in: Atom PDF