Skip to content

refactor: write the Utilities commands through $this->getOutput() - #10675

Open
paulbalandan wants to merge 1 commit into
codeigniter4:4.8from
paulbalandan:utilities-commands-output
Open

paulbalandan wants to merge 1 commit into
codeigniter4:4.8from
paulbalandan:utilities-commands-output

Conversation

@paulbalandan

Copy link
Copy Markdown
Member

Description
First batch of the command migration deferred after #10643. The Utilities commands (env, filter:check, config:check, namespaces, phpini:check, publish, optimize, and routes) now write through $this->getOutput() instead of the static CLI facade. Tables are written with Table::render(), as in the modern-command samples.

No behaviour changes, since AbstractCommand::run() already points the facade at the run's output. The existing tests pass unchanged. The other namespaces follow in their own PRs, and the user guide samples in one docs PR afterwards.

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value (without duplication)
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@carson-codeigniter4 carson-codeigniter4 Bot added 4.8 PRs that target the `4.8` branch. refactor Pull requests that refactor code labels Oct 29, 2026
@paulbalandan
paulbalandan force-pushed the utilities-commands-output branch from c5c2bfc to 19a438e Compare October 29, 2026 07:27
@paulbalandan
paulbalandan force-pushed the utilities-commands-output branch from 19a438e to fa84a28 Compare October 29, 2026 08:51

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4.8 PRs that target the `4.8` branch. refactor Pull requests that refactor code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants