Skip to content

Conversation

@ddevsr
Copy link
Collaborator

@ddevsr ddevsr commented Mar 10, 2025

Description
No needed lowercase event name for logger.

Checklist:

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

@ddevsr ddevsr added the refactor Pull requests that refactor code label Mar 10, 2025
Copy link
Member

@paulbalandan paulbalandan left a comment

Choose a reason for hiding this comment

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

What motivates this change?

@ddevsr
Copy link
Collaborator Author

ddevsr commented Mar 12, 2025

Opps sorry, no needed lower case for logging

@ddevsr ddevsr changed the title refactor: lowercase event name calling twice refactor: remove lowercase event name for logging Mar 12, 2025
@michalsn michalsn merged commit 4b77fd3 into codeigniter4:develop Mar 12, 2025
49 checks passed
@ddevsr ddevsr deleted the patch-3 branch March 12, 2025 08:10
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.

4 participants