Skip to content

style: fix php-cs-fixer.no-header.php#6693

Merged
kenjis merged 2 commits intocodeigniter4:developfrom
kenjis:fix-php-cs-fixer.no-header
Oct 15, 2022
Merged

style: fix php-cs-fixer.no-header.php#6693
kenjis merged 2 commits intocodeigniter4:developfrom
kenjis:fix-php-cs-fixer.no-header

Conversation

@kenjis
Copy link
Copy Markdown
Member

@kenjis kenjis commented Oct 15, 2022

Description

  • fix .php-cs-fixer.no-header.php
  • fix coding style

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

Copy link
Copy Markdown
Member

@MGatner MGatner left a comment

Choose a reason for hiding this comment

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

🚀

Copy link
Copy Markdown
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.

I cannot remember what styling problem I had then but since no problems arise this looks good!

@kenjis kenjis merged commit 44fe2f5 into codeigniter4:develop Oct 15, 2022
@kenjis kenjis deleted the fix-php-cs-fixer.no-header branch October 15, 2022 21:31
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.

3 participants