Skip to content

refactor: write the generator commands through $this->getOutput() - #10690

Merged
paulbalandan merged 1 commit into
codeigniter4:4.8from
paulbalandan:generator-commands-output
Oct 30, 2026
Merged

paulbalandan merged 1 commit into
codeigniter4:4.8from
paulbalandan:generator-commands-output

Conversation

@paulbalandan

Copy link
Copy Markdown
Member

Description
Last batch of the command migration. AbstractGeneratorCommand, make:command, make:controller, make:migration and make:model now write through $this->getOutput().

  • The private methods of AbstractGeneratorCommand that write take the output from their caller. The protected buildPath() keeps its signature.
  • GeneratorTrait is unchanged, since BaseCommand has no getOutput().

No behaviour changes. The existing tests pass unchanged.

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 30, 2026
@paulbalandan
paulbalandan merged commit 9dbf715 into codeigniter4:4.8 Oct 30, 2026
60 checks passed
@paulbalandan
paulbalandan deleted the generator-commands-output branch October 30, 2026 11:44
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