Move check to subscription

Author: cdupuis

datalog/subscription/on_push.edn

@@ -36,6 +36,12 @@
  [?file :git.file/path ?file-path]
  (string-match ?file-path "go\\.mod" _)
 
+ ;; filter out pushes to our atomist/ branches
+ [?ref :git.ref/commit ?commit]
+ [?ref :git.ref/type :git.ref.type/branch]
+ [?ref :git.ref/name ?branch]
+ (not (string-match ?branch "^atomist\\/" _))
+
  ;; check repo filter
  [?commit :git.commit/repo ?repo]
  (skill-repo-selected? ?ctx ?repo)]

lib/events.ts

@@ -14,13 +14,7 @@
  * limitations under the License.
  */
 
-import {
-	EventHandler,
-	github,
-	repository,
-	status,
-	subscription,
-} from "@atomist/skill";
+import { EventHandler, github, repository, subscription } from "@atomist/skill";
 
 export const on_push: EventHandler<
 	[subscription.datalog.OnPush]
@@ -32,10 +26,6 @@ export const on_push: EventHandler<
 		r => r["git.ref/type"]["db/ident"] === "git.ref.type/branch",
 	)?.["git.ref/name"];
 
-	if (!branch || branch.startsWith("atomist/")) {
-		return status.completed(`Ignoring missing or atomist branch`);
-	}
-
 	const p = await ctx.project.clone(
 		repository.gitHub({
 			owner: org["git.org/name"],