Skip to content

refactor: fix ternary.shortNotAllowed errors - #10445

Merged
paulbalandan merged 1 commit into
codeigniter4:developfrom
paulbalandan:short-ternary
Aug 12, 2026
Merged

refactor: fix ternary.shortNotAllowed errors#10445
paulbalandan merged 1 commit into
codeigniter4:developfrom
paulbalandan:short-ternary

Conversation

@paulbalandan

Copy link
Copy Markdown
Member

Description
Using the long form.

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 Aug 12, 2026

@michalsn michalsn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

After a batch of these updates, could you also merge them into 4.8? Letting them accumulate may cause problems later. Thanks!

@paulbalandan

Copy link
Copy Markdown
Member Author

Yes, I'll merge to 4.8 once I got the small wins in.

@paulbalandan
paulbalandan merged commit 4e13c9e into codeigniter4:develop Aug 12, 2026
56 checks passed
@paulbalandan
paulbalandan deleted the short-ternary branch August 12, 2026 14:38
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