all-in-one icon indicating copy to clipboard operation
all-in-one copied to clipboard

Add nextcloud/coding-standard dependency

Open docjyJ opened this issue 1 year ago • 3 comments

Add nextcloud/coding-standard

Attention, all files are affected!

Found 43 of 43 files that can be fixed in 4.645 seconds, 18.00 MB memory used
Script php-cs-fixer fix --dry-run --diff handling the cs:check event returned with error code 8

docjyJ avatar Oct 02 '24 12:10 docjyJ

I split into 2 commits:

  • b3c7cbe added bibliotech
  • cad1a5a run the cs

docjyJ avatar Oct 02 '24 12:10 docjyJ

Thanks again for your contributions @docjyJ! I think this looks good but probably makes sense to wait for e.g. this PR: https://github.com/nextcloud/all-in-one/pull/4804 as otherwise it will be a rebase hell...

szaimen avatar Oct 04 '24 13:10 szaimen

Yes, I know, complicated to merge. Since it's only style, it must not be the worst case.

I dropped the commit from run cs:fix and rebased the branch.

docjyJ avatar Oct 04 '24 18:10 docjyJ