Skip to content

refactor: UX: ClearLogs::execute() error message is misleading after interactive 'n'#10126

Merged
paulbalandan merged 1 commit intocodeigniter4:developfrom
paulbalandan:clear-logs-ux
Apr 22, 2026
Merged

refactor: UX: ClearLogs::execute() error message is misleading after interactive 'n'#10126
paulbalandan merged 1 commit intocodeigniter4:developfrom
paulbalandan:clear-logs-ux

Conversation

@paulbalandan
Copy link
Copy Markdown
Member

Description
In logs:clear, we require --force before the logs can be deleted. in an interactive session and n is provided in the prompt, deletion will be aborted but will display an additional line saying to pass --force. however, the user actually chooses not to pass --force so it is kind of annoying or misleading. it should only be added if we're in non-interactive mode.

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

@paulbalandan paulbalandan added the refactor Pull requests that refactor code label Apr 21, 2026
@paulbalandan paulbalandan merged commit 1e9a2b5 into codeigniter4:develop Apr 22, 2026
55 checks passed
@paulbalandan paulbalandan deleted the clear-logs-ux branch April 22, 2026 02:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Pull requests that refactor code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants