Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Tech: ajout d'un attribut command.run_uid commun aux logs venant d'une même commande #5459

Merged
merged 1 commit into from
Jan 27, 2025
Merged

Tech: ajout d'un attribut command.run_uid commun aux logs venant d'une même commande #5459

merged 1 commit into from
Jan 27, 2025
+59 −1

Conversation

xavfernandez
Copy link
Contributor

🤔 Pourquoi ?

Pour facilement les retrouver & regrouper.

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

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

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

Sorry, something went wrong.

@xavfernandez xavfernandez added the no-changelog Ne doit pas figurer dans le journal des changements. label Jan 24, 2025
@xavfernandez xavfernandez self-assigned this Jan 24, 2025
@xavfernandez xavfernandez force-pushed the xfernandez/group_management_commands_by_request branch from 49bec09 to 89395d3 Compare January 24, 2025 11:26
@xavfernandez xavfernandez changed the title Tech: ajout d'un attribut command.run_uid commun aux logs venant d'une commande Tech: ajout d'un attribut command.run_uid commun aux logs venant d'une même commande Jan 24, 2025
@xavfernandez
utils: add command.run_uid/name to command logs
Loading
Loading status checks…
75fea25 
to easily regroup them
@xavfernandez xavfernandez force-pushed the xfernandez/group_management_commands_by_request branch from 89395d3 to 75fea25 Compare January 24, 2025 13:35
tonial
tonial approved these changes Jan 24, 2025
View reviewed changes
Copy link
Contributor

@tonial tonial left a comment

Choose a reason for hiding this comment

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

💯

@xavfernandez xavfernandez mentioned this pull request Jan 24, 2025
Merged
2 tasks
@xavfernandez xavfernandez added this pull request to the merge queue Jan 27, 2025
Merged via the queue into master with commit 75b9843 Jan 27, 2025
11 checks passed
@xavfernandez xavfernandez deleted the xfernandez/group_management_commands_by_request branch January 27, 2025 08:47
@xavfernandez xavfernandez added tech and removed no-changelog Ne doit pas figurer dans le journal des changements. labels Jan 28, 2025
@leo-naeka leo-naeka added the no-changelog Ne doit pas figurer dans le journal des changements. label Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tonial tonial

@francoisfreitag francoisfreitag

Assignees

@xavfernandez xavfernandez

Labels
no-changelog Ne doit pas figurer dans le journal des changements. tech
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@xavfernandez @tonial @leo-naeka