Merge pull request #295 from jglick/AbstractGitHubNotificationStrategy

AbstractGitHubNotificationStrategy is not an ExtensionPoint

Author: bitwiseman

docs/implementation.adoc

@@ -1,9 +1,7 @@
 = Implementation Guide
 
-== Extension Points
-
-=== AbstractGitHubNotificationStrategy
-This extension points allows traits to modify the contents of a Github status notification for a build.
+== AbstractGitHubNotificationStrategy
+This API allows traits to modify the contents of a GitHub status notification for a build.
 There are currently 3 points in a build lifecycle where notifications are sent:
 
 * On entering the queue
@@ -44,7 +42,6 @@ explicitly apply a `DefaultGitHubNotificationStrategy` to the source context in
 
 Duplicate (by equality) strategies are ignored when applied to the source context.
 
-==== Implementations:
-https://github.com/jenkinsci/github-scm-trait-notification-context-plugin[github-scm-trait-notification-context]
-
+=== Implementations:
 
+https://github.com/jenkinsci/github-scm-trait-notification-context-plugin[github-scm-trait-notification-context]

src/main/java/org/jenkinsci/plugins/github_branch_source/AbstractGitHubNotificationStrategy.java

@@ -24,7 +24,6 @@
 
 package org.jenkinsci.plugins.github_branch_source;
 
-import hudson.ExtensionPoint;
 import hudson.model.TaskListener;
 
 import java.util.List;
@@ -33,7 +32,7 @@
  * Represents a strategy for constructing GitHub status notifications
  * @since 2.3.2
  */
-public abstract class AbstractGitHubNotificationStrategy implements ExtensionPoint {
+public abstract class AbstractGitHubNotificationStrategy {
 
     /**
      * Creates the list of {@link GitHubNotificationRequest} for the given context.