WPCS improvements to the Admin Component Options

[espellcaste]

While working on the list command for wp-cli-buddypress, I sensed bp_core_admin_components_options could be improved. Patch eminent. :)

[DJPaul]

This looks ok but if you got this from running PHPCS/PHPCBF, I am up for an effort like ​#WP41057 on our codebase -- after @r-a-y lands #7218 otherwise his patches are going to be a nightmare to update. :(

[espellcaste]

Makes total sense! I'm closing this now as invalid and I'll think about doing what was done on ​#WP41057.

Thank you for the feedback Paul. :)

[DJPaul]

The patch is easy (really, just copy in WP's PHPCS file), the issue is reviewing every single change (there's thousands of changes) and checking we agree with the relevant coding standard, while not breaking any sizeable patch still in progress. Right now, I only know of Ray's set of patches that would be seriously inconvenienced by doing this.

I will probably start doing it piecemeal - I might do Nouveau first, before I dive into another round of code reviews on that.

[netweb]

Yup, pretty much that, I've not started on this either as waiting for WPCS 1.0 to be released rather than playing with branches in composer.json, once WPCS 1.0 is released we can start seeing how things look_,

Was that some trailing commas I wrote, @jjj has opinions on this, i.e, something about someplace freezing over before these get committed in the bb's... a couple more tweaks will also be required that I can't remember off the top of my head right now

[johnjamesjacoby]

I don’t want this to turn into the JJJ styleguide, but it’s obviously true that I have strong opinions on how elegant code is best written, and I’ve been irresponsible by not publishing those someplace to make them more clear.

In short, it’s like having a nice signature – it’s almost meaningless until people compliment you on it, but you won’t want to change it once they start, and there’s no other real reason to practice it when the reality is you can just sign an X.

We can build tools to recreate coding standards, but that will never beat the zen-like practice of strolling through code and reflecting on every nuanced character choice.

It’s an exercise that isn’t appealing to everyone, but it’s important to me, and an important part of what I selfishly enjoy teaching others.