Skip to content

refactor: further rename --handler to --sort-by-handler for routes#10125

Open
paulbalandan wants to merge 1 commit intocodeigniter4:developfrom
paulbalandan:rename-further-h-option
Open

refactor: further rename --handler to --sort-by-handler for routes#10125
paulbalandan wants to merge 1 commit intocodeigniter4:developfrom
paulbalandan:rename-further-h-option

Conversation

@paulbalandan
Copy link
Copy Markdown
Member

Description
Follow up to #10113 .

I realized --handler is quite ambiguous. It could mean the option is requiring a value (the handler) to be passed but in reality is a flag.

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

@michalsn
Copy link
Copy Markdown
Member

We need to update user guide too: https://github.com/codeigniter4/CodeIgniter4/blob/develop/user_guide_src/source/incoming/routing.rst?plain=1#L1008

@paulbalandan paulbalandan force-pushed the rename-further-h-option branch from 3469913 to 460eb14 Compare April 20, 2026 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants