Projet

Général

Profil

Bug #63321

Fichiers manquants dans le .gitignore

Ajouté par Corentin Séchet il y a environ 2 ans. Mis à jour il y a environ 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
29 mars 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

.coverage
applications/
build/
coverage-py3-django22-coverage-authentic.xml
coverage-py3-django22-coverage-hobo.xml
coverage-py3-django22-coverage-multipublik.xml
coverage-py3-django22-coverage-multitenant.xml
coverage-py3-django22-coverage-passerelle.xml
coverage-py3-django22-coverage-schemas.xml
debug.log
junit-py3-django22-coverage-authentic.xml
junit-py3-django22-coverage-hobo.xml
junit-py3-django22-coverage-multipublik.xml
junit-py3-django22-coverage-multitenant.xml
junit-py3-django22-coverage-passerelle.xml
junit-py3-django22-coverage-schemas.xml


Fichiers


Demandes liées

Lié à Hobo - Bug #63349: tests/test_application.py, ne pas créer de fichier applications/whatever dans le répertoire du checkoutNouveau30 mars 2022

Actions

Révisions associées

Révision 8463aa66 (diff)
Ajouté par Corentin Séchet il y a environ 2 ans

trivial: add missing files in .gitignore (#63321)

Historique

#1

Mis à jour par Corentin Séchet il y a environ 2 ans

#2

Mis à jour par Nicolas Roche il y a environ 2 ans

Autant ajouter les répertoires de couverture de code (htmlcov-py3-django22-coverage-hobo/ and co) ?

#3

Mis à jour par Corentin Séchet il y a environ 2 ans

  • Fichier 0001-trivial-add-missing-files-in-.gitignore-63321.patch ajouté
#4

Mis à jour par Corentin Séchet il y a environ 2 ans

  • Fichier 0001-trivial-add-missing-files-in-.gitignore-63321.patch supprimé
#5

Mis à jour par Corentin Séchet il y a environ 2 ans

  • Fichier 0001-trivial-add-missing-files-in-.gitignore-63321.patch ajouté
#6

Mis à jour par Corentin Séchet il y a environ 2 ans

  • Fichier 0001-trivial-add-missing-files-in-.gitignore-63321.patch supprimé
#8

Mis à jour par Corentin Séchet il y a environ 2 ans

La troisième c'est la bonne

#9

Mis à jour par Corentin Séchet il y a environ 2 ans

Alors en fait c'est la quatrième : les dossiers htmlcov-* contiennent déjà un .gitignore généré par coverage, pas la peine de les ajouter ici.

#10

Mis à jour par Thomas Noël il y a environ 2 ans

Je suis étonné par les items qui commencent par un /, on n'en a pas ailleurs. On a plutôt "build/" que "/build/". Un peu la flemme de lire "man gitignore", tu peux dire pourquoi tu as mis ça ? (c'est peut-être toi qui fait bien et nous un peu n'imp depuis longtemps)

#11

Mis à jour par Frédéric Péters il y a environ 2 ans

Ça dit que ça concerne la racine du projet, on a ça aussi un peu plus haut, /hobo.egg-info, dans l'absolu souvent les noms de fichiers sont uniques, on ne trouvera pas un répertoire hobo.egg-info ailleurs, donc ça n'a pas grande importance. (ailleurs, le / à la fin dirait que ça doit matcher uniquement un répertoire, ce qui souvent pareil n'est pas nécessaire, on ne trouvera pas de fichier nommé hobo.egg-info).

(par contre ici applications je ne vois pas comment il se crée et existe à la racine, c'est à mon avis une particularité locale, qui ne devrait pas être mise ici).

#12

Mis à jour par Emmanuel Cazenave il y a environ 2 ans

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

(par contre ici applications je ne vois pas comment il se crée et existe à la racine, c'est à mon avis une particularité locale, qui ne devrait pas être mise ici).

C'est idem chez moi, pas cherché à comprendre comment ça arrivait là.

#13

Mis à jour par Nicolas Roche il y a environ 2 ans

C'est #60699.

$ tox -e py3-django22-coverage-hobo -- tests/test_application.py::test_create_application

#14

Mis à jour par Frédéric Péters il y a environ 2 ans

Il s'agirait de revoir ça pour que ça s'écrive dans un répertoire sous /tmp/ je dirais, plutôt que cacher l'affaire ici.

#15

Mis à jour par Frédéric Péters il y a environ 2 ans

  • Lié à Bug #63349: tests/test_application.py, ne pas créer de fichier applications/whatever dans le répertoire du checkout ajouté
#16

Mis à jour par Corentin Séchet il y a environ 2 ans

Fred a tout répondu :) Pour applications, le slash du début était nécessaire, puisqu'il y a un autre dossier applications dans les sources. Pour build ça se discute, mais je trouve plus sûr d'ignorer précisément ce qu'il faut plutôt que des fichiers soient oubliés dans un commit à cause d'un gitignore un peu trop large.

#17

Mis à jour par Thomas Noël il y a environ 2 ans

  • Statut changé de Solution proposée à Solution validée

Ca me va !

#18

Mis à jour par Corentin Séchet il y a environ 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 8463aa66f4d554b8072f52adf54e9532ec07d2bd
Author: Corentin Séchet <csechet@entrouvert.com>
Date:   Tue Mar 29 17:30:14 2022 +0200

    trivial: add missing files in .gitignore (#63321)
#19

Mis à jour par Transition automatique il y a environ 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#20

Mis à jour par Transition automatique il y a presque 2 ans

Automatic expiration

Formats disponibles : Atom PDF