Project

General

Profile

Development #91730

nox: poser reuse_venv=True par défaut

Added by Benjamin Dauvergne about 1 month ago. Updated about 1 month ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
12 June 2024
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

Ça correspond à ce que faisait tox avant.

Associated revisions

Revision c6408d3a (diff)
Added by Benjamin Dauvergne about 1 month ago

ci: add reuse_venv=True to all sessions (#91730)

History

#2

Updated by Benjamin Dauvergne about 1 month ago

Valentin Deniaud a écrit :

Plutôt non https://git.entrouvert.org/entrouvert/combo/pulls/265#issuecomment-36705

J'en suis à me demander si ce n'est pas trop magique : si par hasard je Ctrl + C au moment de lancer nox, l'environnement installé est incomplet et tous les futurs lancements vont crasher, jusqu'à ce que je trouve la longue option --no-reuse-existing-virtualenvs. Peut-être qu'il faudrait juste s'assurer de très bien documenter l'option -R à passer à nox ?

C'était exactement pareil avec tox donc plutôt oui. Merci de ne pas changer mon environnement de travail sans me demander mon avis.

#3

Updated by Valentin Deniaud about 1 month ago

Je m'autoréponds de manière plus constructive,

Valentin Deniaud a écrit :

J'en suis à me demander si ce n'est pas trop magique : si par hasard je Ctrl + C au moment de lancer nox, l'environnement installé est incomplet et tous les futurs lancements vont crasher, jusqu'à ce que je trouve la longue option --no-reuse-existing-virtualenvs. Peut-être qu'il faudrait juste s'assurer de très bien documenter l'option -R à passer à nox ?

Ici je mélange les deux options cachées derrière -R, ce n'est pas --no-reuse-existing-virtualenvs qui provoque le crash des futurs lancements, c'est --no-install. Pas d'inconvénient à réutiliser les venv par défaut donc.

#4

Updated by Benjamin Dauvergne about 1 month ago

Valentin Deniaud a écrit :

Je m'autoréponds de manière plus constructive,

Merci beaucoup.

#5

Updated by Benjamin Dauvergne about 1 month ago

  • Assignee set to Benjamin Dauvergne
#6

Updated by Robot Gitea about 1 month ago

  • Status changed from Nouveau to En cours

Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :

#7

Updated by Robot Gitea about 1 month ago

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

Updated by Robot Gitea about 1 month ago

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

Valentin Deniaud (vdeniaud) a approuvé une pull request sur Gitea concernant cette demande :

#9

Updated by Robot Gitea about 1 month ago

  • Status changed from Solution validée to Résolu (à déployer)

Benjamin Dauvergne (bdauvergne) a mergé une pull request sur Gitea concernant cette demande :

#10

Updated by Transition automatique about 1 month ago

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

Also available in: Atom PDF