joomla-cms icon indicating copy to clipboard operation
joomla-cms copied to clipboard

[6.0] Admin modules code style

Open brianteeman opened this issue 1 year ago • 4 comments

various files are excluded from the automated checks // Ignore template files as PHP CS fixer can't handle them properly // https://github.com/PHP-CS-Fixer/PHP-CS-Fixer/issues/3702#issuecomment-396717120

Summary of Changes

This is a semi-manual review of the site modules to apply

  • trailing_comma_in_multiline
  • native_function_invocation
  • single_line_after_imports

Testing Instructions

code review

Link to documentations

Please select:

  • [ ] Documentation link for docs.joomla.org:

  • [x] No documentation changes for docs.joomla.org needed

  • [ ] Pull Request link for manual.joomla.org:

  • [x] No documentation changes for manual.joomla.org needed

brianteeman avatar Apr 14 '24 18:04 brianteeman

j4i this marks all layouts as changed on update

HLeithner avatar Apr 24 '24 07:04 HLeithner

j4i this marks all layouts as changed on update

??

brianteeman avatar Apr 24 '24 08:04 brianteeman

This pull request has been automatically rebased to 5.2-dev.

HLeithner avatar Apr 24 '24 09:04 HLeithner

rebased as requested

brianteeman avatar Apr 24 '24 19:04 brianteeman

closed due to lack of interest

brianteeman avatar May 30 '24 21:05 brianteeman