Add a dedicated annotation for aggregator Plugins

Cool, thanks:)
That means we can get rid of

$info = $this->getDefinition();
..
$form['processors'][$info['id']] = array();

for $this->id?

in \Drupal\aggregator\Plugin\aggregator\processor\DefaultProcessor::settingsForm()

EDIT: nvm, this is totally wrong:)

its good to go:)

+++ b/core/modules/aggregator/lib/Drupal/aggregator/Plugin/Annotation/AggregatorFetcher.phpundefined
@@ -0,0 +1,43 @@
+namespace Drupal\aggregator\Plugin\Annotation;

+++ b/core/modules/aggregator/lib/Drupal/aggregator/Plugin/Annotation/AggregatorParser.phpundefined
@@ -0,0 +1,43 @@
+namespace Drupal\aggregator\Plugin\Annotation;

+++ b/core/modules/aggregator/lib/Drupal/aggregator/Plugin/Annotation/AggregatorProcessor.phpundefined
@@ -0,0 +1,43 @@
+namespace Drupal\aggregator\Plugin\Annotation;

While we're only using them for plugins, Annotations are technically unrelated to that subsystem (yeah, seriously), and should just be in their own namespace: \Drupal\$module\Annotation

+++ b/core/modules/aggregator/lib/Drupal/aggregator/Plugin/Annotation/AggregatorFetcher.phpundefined
@@ -0,0 +1,43 @@
+   * The string should be wrapped in a @Translatable()
...
+   * The string should be wrapped in a @Translatable()

+++ b/core/modules/aggregator/lib/Drupal/aggregator/Plugin/Annotation/AggregatorParser.phpundefined
@@ -0,0 +1,43 @@
+   * The string should be wrapped in a @Translatable()
...
+   * The string should be wrapped in a @Translatable()

+++ b/core/modules/aggregator/lib/Drupal/aggregator/Plugin/Annotation/AggregatorProcessor.phpundefined
@@ -0,0 +1,43 @@
+   * The string should be wrapped in a @Translatable()
...
+   * The string should be wrapped in a @Translatable()

I'm not sure that this is necessary to repeat for every property. But if it is, we should probably just do @see \Drupal\Core\Annotation\Translatable or something similar.

I could only find some small nitpicks here, otherwise looks great.

+++ b/core/modules/aggregator/lib/Drupal/aggregator/Annotation/AggregatorFetcher.php
@@ -0,0 +1,43 @@
+ * Contains \Drupal\aggregator\Annotation\AggregatorFetcher.
+ */
+namespace Drupal\aggregator\Annotation;

Missing empty line.

+++ b/core/modules/aggregator/lib/Drupal/aggregator/Annotation/AggregatorFetcher.php
@@ -0,0 +1,43 @@
+}
+

Extra empty line at the end of the file?

The two above apply to all new class files, not just AggregatorFetcher.

+++ b/core/modules/aggregator/lib/Drupal/aggregator/Annotation/AggregatorFetcher.php
@@ -0,0 +1,43 @@
+   * @ingroup plugin_translatable
+   *
+   * @var \Drupal\Core\Annotation\Translation

@ingroup should be below @var.

All good now.

Committed to 8.x. Thanks.