Project

General

Profile

Development #90625

nouvelle version de pylint

Added by Yann Weber 9 days ago. Updated 3 days ago.

Status:
En cours
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
15 May 2024
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

Pylint 3.2.0 se retrouve utilisé dans notre pipeline jenkins.

De nouveaux warnings sont levés, principalement avec l'introduction de possibly-used-before-assignment

En plus :

Pour ce qui est des warnings levés par le nouveau possibly-used-before-assignment la plupart des exemples que j'ai regardé jusqu'à présent sont cohérent et pertinents avec la description du warning : when relying on names after an if/else switch when one branch failed to define the name, raise, or return

Un des faux positifs sur lesquels je suis tombé est https://jenkins.entrouvert.org/job/gitea/job/authentic/job/wip%252F89456-api-client-password/14/pylint/folder.2098948229/source.d2fa48c9-8f86-423a-aace-a8721c3ab184/#116

On a bien "une branche qui échoue a définir TenantMiddleware" à la ligne 44, un if MULTITENANT sans else qui fait l'import, mais le soucis c'est que le warning est levé sur un bout de code qui est dans un if MULTITENANT :/

Une solution pourrait être de désactiver possibly-used-before-assignment dans pylint.rc ?


Related issues

Related to Authentic 2 - Bug #90624: Nouvelle version de pylint qui casse l’intégration continueSolution déployée15 May 2024

Actions
Related to w.c.s. - Development #90582: build, adapter à nouvelle version de pylintSolution déployée14 May 2024

Actions
Related to Authentic 2 - Bug #90834: Nouvelle version de pylint qui casse l’intégration continueRésolu (à déployer)21 May 2024

Actions

History

#1

Updated by Yann Weber 9 days ago

  • Related to Bug #90624: Nouvelle version de pylint qui casse l’intégration continue added
#2

Updated by Yann Weber 9 days ago

#3

Updated by Robot Gitea 9 days ago

  • Status changed from Nouveau to En cours
  • Assignee set to Yann Weber

Yann Weber (yweber) a ouvert une pull request sur Gitea concernant cette demande :

#4

Updated by Yann Weber 3 days ago

  • Related to Bug #90834: Nouvelle version de pylint qui casse l’intégration continue added
#5

Updated by Robot Gitea 3 days ago

  • Status changed from En cours to Solution proposée
#6

Updated by Robot Gitea 3 days ago

  • Status changed from Solution proposée to En cours

Benjamin Dauvergne (bdauvergne) a relu et demandé des modifications sur une pull request sur Gitea concernant cette demande :

Also available in: Atom PDF