Development #63191
Retours pylint - enlever disable=no-value-for-parameter
Début:
25 mars 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
elif xmltype.has_simple_content(): base_schema = cls.type_to_jsonschema(xmltype.base_type) if not xmltype.attributes: schema = base_schema else: cls.attributegroup_to_jsonschema(xmltype.attributes) # pylint: disable=no-value-for-parameter schema['properties']['$'] = base_schema return schema
Corriger ce bout de code
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Lauréline Guérin il y a environ 2 ans
- Lié à Development #62099: Faire planter le build jenkins sur une erreur pylint ajouté
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Assigné à mis à Benjamin Dauvergne
Benjamin, c'est toi qui avais écrit ce code à l'origine, tu veux regarder ?
De mémoire, pylint râle parce que les paramètres d'appel ne correspondent pas à la signature de la méthode; ce code n'est pas couvert par les tests
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
- Fichier 0001-utils-xml-fix-parsing-of-simpleContent-XML-schema-63.patch 0001-utils-xml-fix-parsing-of-simpleContent-XML-schema-63.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Avec un bout de test au passage, c'était effectivement foireux mais ma passion pour les schémas XML a ses limites. Les XSD qu'on avait ne contiennent pas cette construction.
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit fb7fcf5f883504bfc98b904430e016bbc13affed Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Thu Jul 7 13:36:57 2022 +0200 utils/xml: fix parsing of simpleContent XML schema (#63191)
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Résolu (à déployer) à En cours
J'ai bien corrigé le code mais je n'ai pas réactivé no-value-for-parameter.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de En cours à Résolu (à déployer)
Ah si c'était en commentaire.
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
utils/xml: fix parsing of simpleContent XML schema (#63191)