Convert MigrateDestination plugin discovery to attributes

Note that change to `core/modules/migrate/src/Plugin/MigratePluginManager.php` is also needed by other migrate plugins.

The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Rebased.

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.

Searched for @MigrateDestination and all 31 instances in core have been replaced.

Checked the attribute for the deriver class and no @Annotation.

Believe this one is good.

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

Re-opening because the balance of destination plugins outside a couple done in 📌 Update MigratePluginManager to include both attribute and annotation class Fixed still need to be converted. MR here has been rebased and is ready for review.

Believe 2 were missed EntityFile and Book.php

Believe 2 were missed EntityFile and Book.php

Those two were already done in 📌 Update MigratePluginManager to include both attribute and annotation class Fixed
https://git.drupalcode.org/project/drupal/-/blob/11.x/core/modules/file/...
https://git.drupalcode.org/project/drupal/-/blob/11.x/core/modules/book/...

You are correct, that's my mistake

Rebase my local and applying MR all instances of @MigrateDestination have been replaced.

Committed and pushed 38e722aa6f to 11.x and 3f64e2bf2c to 10.3.x. Thanks!

quietone → closed merge request !6798

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