Skip to content
This repository has been archived by the owner on Jun 1, 2022. It is now read-only.

Update brigade.js to use the GitHub library #20

Merged
merged 1 commit into from
Aug 12, 2019

Conversation

radu-matei
Copy link
Contributor

Note that this PR does not handle the issue of actually testing itself on each pull request, but just replaces the existing GitHub notification logic with the one in the utils library.

Copy link
Contributor

@vdice vdice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏 Ooooh so streamlined, so clean! A few comments/questions.

brigade.js Outdated Show resolved Hide resolved
Signed-off-by: Radu M <root@radu.sh>

Fix typo for issue_comment:edited handler in CI

Signed-off-by: Radu M <root@radu.sh>
@vdice
Copy link
Contributor

vdice commented Aug 5, 2019

/brig run

@radu-matei radu-matei merged commit 2f1e90a into brigadecore:master Aug 12, 2019
@radu-matei radu-matei deleted the brigade-utils-github branch August 12, 2019 20:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants