Make WordPress Core

Opened 9 years ago

Closed 9 years ago

#34801 closed enhancement (fixed)

Remove debug from default wp-cli.yml

Reported by: rodrigosprimo's profile rodrigosprimo Owned by:
Milestone: 4.4 Priority: normal
Severity: normal Version:
Component: Build/Test Tools Keywords: has-patch
Focuses: Cc:

Description

Since version 0.21.0, WP-CLI is using the debug flag for more verbosity during its bootstrap process (see https://github.com/wp-cli/wp-cli/pull/2138) thus I suggest that this flag is removed from the wp-cli.yml file used for core development.

After this change with this flag enabled every WP-CLI command outputs information that is useful only if you are debugging the execution of the tool. This includes the auto complete feature which makes it hard to use. For example, see the output below if I type wp plugin de and try to use auto complete:

$ wp plugin deDebug: Using default global config: /home/rodrigo/.wp-cli/config.yml (0.008s)
Debug: Using project config: /home/rodrigo/devel/wordpress-develop/wp-cli.yml (0.008s)
Debug: ABSPATH defined: /home/rodrigo/devel/wordpress-develop/src/ (0.019s)
Debug: Running command: cli completions (0.019s)
  wp plugin de
deactivate   delete

When debug is disabled the output is:

$ wp plugin de
deactivate   delete     

Attachments (1)

34801.patch (140 bytes) - added by rodrigosprimo 9 years ago.

Download all attachments as: .zip

Change History (5)

@rodrigosprimo
9 years ago

#1 @johnbillion
9 years ago

  • Milestone changed from Awaiting Review to 4.4

#2 @swissspidy
9 years ago

  • Keywords has-patch added

#3 @danielbachhuber
9 years ago

For the next person to come across this ticket, you can also disable a flag by prefixing with --no- (e.g. --no-debug)

#4 @johnbillion
9 years ago

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in [35746].

Note: See TracTickets for help on using tickets.