Project

General

Profile

Développement #69709

pre-commit-hooks, utiliser djhtml

Added by Valentin Deniaud about 2 years ago. Updated about 2 years ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
29 September 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Je suis en train d'appliquer ça partout mais ici ça fait planter un test, ticket donc.


Files

0002-misc-add-djhtml-files-notes-69422.patch (1.82 KB) 0002-misc-add-djhtml-files-notes-69422.patch Valentin Deniaud, 29 September 2022 02:17 PM
0001-misc-apply-djhtml-69422.patch (327 KB) 0001-misc-apply-djhtml-69422.patch Valentin Deniaud, 29 September 2022 02:17 PM

Related issues

Related to Publik - Développement #69422: pre-commit-hooks, utiliser djhtml​ partoutFermé22 September 2022

Actions

Associated revisions

Revision 4784a399 (diff)
Added by Valentin Deniaud about 2 years ago

misc: apply djhtml (#69709)

Revision 54126b6c (diff)
Added by Valentin Deniaud about 2 years ago

misc: add djhtml files/notes (#69709)

History

#1

Updated by Valentin Deniaud about 2 years ago

#2

Updated by Valentin Deniaud about 2 years ago

Le reformattage a ajouté un espace quelque part dans combo/wcs/card.html et il y avait un test sensible aux espaces, donc modif nécessaire

-    assert cell_resp.text.replace('\n', '') == ''  # empty-cell
+    assert cell_resp.text.replace('\n', '').strip() == ''  # empty-cell

#3

Updated by Emmanuel Cazenave about 2 years ago

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

Updated by Valentin Deniaud about 2 years ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 54126b6ccf165a324d473952cbe0bfd617050fdc
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Thu Sep 29 11:50:30 2022 +0200

    misc: add djhtml files/notes (#69709)

commit 4784a3990eec0aadadce99bf1da21c0b531b289d
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Thu Sep 29 11:51:33 2022 +0200

    misc: apply djhtml (#69709)
#5

Updated by Transition automatique about 2 years ago

  • Status changed from Résolu (à déployer) to Solution déployée
#6

Updated by Transition automatique almost 2 years ago

Automatic expiration

Also available in: Atom PDF