Convert MigrateProcessPlugin plugin discovery to attributes

Note: BC approach to handle annotations and attributes in MigratePluginManager class is being evaluated in https://www.drupal.org/project/drupal/issues/3424509 📌 Update MigratePluginManager to include both attribute and annotation class Fixed

We should revisit and rebase this PR after this is resolved.

Attaching a patch that has conversions for the process plugins. It does not include work on the manager or the attribute class.

Blocked on 📌 Update MigratePluginManager to include both attribute and annotation class Fixed

Re-opened to convert the remaining process plugins.

Seems all 78 instances of @MigrateProcessPlugin have been replaced.

I am disappointed to see many of these changed to one-liners and I have noticed this on other issues as well. It seems reasonable to me to keep the style of the coding standard for annotations. And worse, for me, it makes these harder to scan. Since there is no coding standard specific for attributes I have created #3439004: Coding standard for attributes → .

I am disappointed to see many of these changed to one-liners and I have noticed this on other issues as well.

I received feedback to convert id-only attributes to one-liners on other issues for consistency, so I changed them similarly here. I agree that multi-line attributes with named properties is more readable, but people have made cases the other way. While I do have a preference, I don't feel that strongly about it, but putting coding standards in place is a reasonable next step.

Committed and pushed 8c0975a467 to 11.x and ee650b2e7f to 10.3.x. Thanks!

quietone → closed merge request !7316

Automatically closed - issue fixed for 2 weeks with no activity.