Skip to content

PB-2094: Use CustomBaseCommand for all management commands#620

Merged
msom merged 1 commit into
developfrom
feat-PB-2094-command-base
Dec 3, 2025
Merged

PB-2094: Use CustomBaseCommand for all management commands#620
msom merged 1 commit into
developfrom
feat-PB-2094-command-base

Conversation

@msom

@msom msom commented Dec 2, 2025

Copy link
Copy Markdown
Contributor

Currently, only a few of all Django management commands inherit from CustomBaseCommand (which adds the --logger option).

This PR:

  • Changes command inheritance to consistently use CustomBaseCommand
  • Replaces print with self.print
  • Removes unused loggers
  • Removes the color-formatting for log outputs (this makes no sense)

@msom msom requested review from benschs and ltshb December 2, 2025 06:35
@msom msom force-pushed the feat-PB-2094-command-base branch from af9f029 to a9de882 Compare December 2, 2025 06:36

@ltshb ltshb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice thanks

@msom msom merged commit db6d5ad into develop Dec 3, 2025
3 checks passed
@msom msom deleted the feat-PB-2094-command-base branch December 3, 2025 09:46
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.

2 participants