Skip to content

Tech: importer les librairies frequemment utilisées dans shell_plus#5803

Merged
vincentporte merged 1 commit into
masterfrom
vp/shell_plus_imports
Mar 25, 2025
Merged

Tech: importer les librairies frequemment utilisées dans shell_plus#5803
vincentporte merged 1 commit into
masterfrom
vp/shell_plus_imports

Conversation

@vincentporte
Copy link
Copy Markdown

🤔 Pourquoi ?

Eviter le copier-coller de snippets dans shell_plus

🍰 Comment ?

Ajout des libs dans la variable d'environnement SHELL_PLUS_IMPORTS dans settings.dev

🚨 À vérifier

NON Mettre à jour le CHANGELOG_breaking_changes.md ?
NON Ajouter l'étiquette « Bug » ?

Copy link
Copy Markdown
Contributor

@EwenKorr EwenKorr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je n'utilise pas si fréquemment les dates dans shell_plus mais le coût me semble nul alors c'est une bonne idée selon moi ! :)

Comment thread config/settings/dev.py Outdated
Copy link
Copy Markdown
Contributor

@xavfernandez xavfernandez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pareil, je n'utilise pas (mais il faudrait sûrement :) ) et plutôt pour l'import du module datetime à la place des symboles.

@leo-naeka
Copy link
Copy Markdown
Contributor

Idem, je n'utilise pas, +1 sur datetime

Copy link
Copy Markdown
Contributor

@celine-m-s celine-m-s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super idée !

@vincentporte vincentporte force-pushed the vp/shell_plus_imports branch from c214d46 to 4d4815c Compare March 25, 2025 15:15
@vincentporte vincentporte enabled auto-merge March 25, 2025 15:16
@vincentporte vincentporte added this pull request to the merge queue Mar 25, 2025
Merged via the queue into master with commit 3e988ad Mar 25, 2025
@vincentporte vincentporte deleted the vp/shell_plus_imports branch March 25, 2025 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants