Compare commits
8 Commits
ec6edf598c
...
v0.1.0
| Author | SHA1 | Date | |
|---|---|---|---|
| 87f4a94734 | |||
| a28d6a2509 | |||
| ced41f4a49 | |||
| c117cb847d | |||
| d1cfd7c20b | |||
| b6b5c27ffc | |||
| def95cd838 | |||
| 79f21f8a54 |
3
AUTHORS.md
Normal file
3
AUTHORS.md
Normal file
@@ -0,0 +1,3 @@
|
|||||||
|
### Authors
|
||||||
|
|
||||||
|
- joe fleming ([w33ble](https://github.com/w33ble))
|
||||||
8
CHANGELOG.md
Normal file
8
CHANGELOG.md
Normal file
@@ -0,0 +1,8 @@
|
|||||||
|
### Changelog
|
||||||
|
|
||||||
|
#### 0.1.0 (18 October 2018)
|
||||||
|
- docs: update readme, add sample env [`a28d6a2`](https://git.w33ble.com/w33ble/gh-action-bot/commit/a28d6a25096efb9c8acea3e8d26e7f7f50dfd6b5)
|
||||||
|
- feat: add retest comment, delete it [`ced41f4`](https://git.w33ble.com/w33ble/gh-action-bot/commit/ced41f4a495823205e5769e4531f923a982d29b4)
|
||||||
|
- feat: process 2 pages of events [`c117cb8`](https://git.w33ble.com/w33ble/gh-action-bot/commit/c117cb847d5cb7a6b57b705e0c192cc725c0d646)
|
||||||
|
- feat: fetch data for prs that need retest [`d1cfd7c`](https://git.w33ble.com/w33ble/gh-action-bot/commit/d1cfd7c20b17802f2de1b9153ae9a8392ed1bc44)
|
||||||
|
- feat: add history tracking [`b6b5c27`](https://git.w33ble.com/w33ble/gh-action-bot/commit/b6b5c27ffc43ff2e1c1ccf041958021cf7c8c4d4)
|
||||||
12
README.md
12
README.md
@@ -1,10 +1,16 @@
|
|||||||
# gh-action-bot
|
# gh-action-bot
|
||||||
|
|
||||||
Action bot for Github.
|
Action bot for Github. Checks a repo's event stream and leaves a ghost comment when your criteria is met.
|
||||||
|
|
||||||
[](https://raw.githubusercontent.com/w33ble/gh-action-bot/master/LICENSE)
|
[](https://raw.githubusercontent.com/w33ble/gh-action-bot/master/LICENSE)
|
||||||
[](https://www.npmjs.com/package/gh-action-bot)
|
[](https://nodejs.org/api/documentation.html#documentation_stability_index)
|
||||||
[](https://nodejs.org/api/documentation.html#documentation_stability_index)
|
|
||||||
|
## Usage
|
||||||
|
|
||||||
|
`node . owner/repo`
|
||||||
|
|
||||||
|
You'll need to set some environment flags. Check `sample-env` for what is available, and you can copy the file to `.env` in the root and add your settings there as well.
|
||||||
|
|
||||||
|
|
||||||
#### License
|
#### License
|
||||||
|
|
||||||
|
|||||||
@@ -1,6 +1,7 @@
|
|||||||
{
|
{
|
||||||
"name": "gh-action-bot",
|
"name": "gh-action-bot",
|
||||||
"version": "0.0.0",
|
"version": "0.1.0",
|
||||||
|
"private": true,
|
||||||
"description": "Action bot for Github",
|
"description": "Action bot for Github",
|
||||||
"main": "index",
|
"main": "index",
|
||||||
"module": "index.mjs",
|
"module": "index.mjs",
|
||||||
|
|||||||
6
sample-env
Normal file
6
sample-env
Normal file
@@ -0,0 +1,6 @@
|
|||||||
|
GITHUB_ACCESS_TOKEN=xxx
|
||||||
|
COMMENT_ACTOR=ci-comment-username
|
||||||
|
COMMENT_BODY_REGEXP=build failed
|
||||||
|
COMMENT_BODY_REGEXP_FLAGS=i
|
||||||
|
PULL_LABEL_FILTER=
|
||||||
|
PULL_RETEST_BODY=retest
|
||||||
@@ -1,28 +1,43 @@
|
|||||||
|
import dotenv from 'dotenv';
|
||||||
import createRepo from './lib/create_repo.mjs';
|
import createRepo from './lib/create_repo.mjs';
|
||||||
import getComments from './lib/get_comments.mjs';
|
import getComments from './lib/get_comments.mjs';
|
||||||
import getPull from './lib/get_pull.mjs';
|
import getPull from './lib/get_pull.mjs';
|
||||||
import getCommits from './lib/get_commits.mjs';
|
import getCommits from './lib/get_commits.mjs';
|
||||||
import getCommitStatus from './lib/get_commit_status.mjs';
|
import getCommitStatus from './lib/get_commit_status.mjs';
|
||||||
|
import { createComment, deleteComment } from './lib/comments.mjs';
|
||||||
import History from './lib/history.mjs';
|
import History from './lib/history.mjs';
|
||||||
|
|
||||||
|
// load env vars from .env file
|
||||||
|
dotenv.config();
|
||||||
|
|
||||||
export default async function() {
|
export default async function() {
|
||||||
// parse repo name from cli and create repo instance
|
// parse repo name from cli and create repo instance
|
||||||
const repo = createRepo(process.argv.splice(2)[0]);
|
const repo = createRepo(process.argv.splice(2)[0]);
|
||||||
|
const {
|
||||||
|
COMMENT_BODY_REGEXP,
|
||||||
|
COMMENT_BODY_REGEXP_FLAGS,
|
||||||
|
COMMENT_ACTOR,
|
||||||
|
PULL_LABEL_FILTER,
|
||||||
|
PULL_RETEST_BODY,
|
||||||
|
} = process.env;
|
||||||
|
|
||||||
// load the history module
|
// load the history module
|
||||||
const history = new History();
|
const history = new History();
|
||||||
|
|
||||||
// fetch comment info from event stream, filter for only new comments
|
// fetch comment info from event stream, filter for only new comments
|
||||||
const comments = (await getComments(repo, {
|
const comments = (await getComments(repo, {
|
||||||
body: /build failed/i,
|
body: new RegExp(COMMENT_BODY_REGEXP, COMMENT_BODY_REGEXP_FLAGS),
|
||||||
actor: 'elasticmachine',
|
actor: COMMENT_ACTOR,
|
||||||
})).filter(comment => !history.get(comment.id));
|
})).filter(comment => !history.get(comment.id));
|
||||||
|
|
||||||
// read pull data, filter out any closed pulls
|
// read pull data
|
||||||
const pulls = (await Promise.all(
|
const pulls = (await Promise.all(
|
||||||
comments.map(async comment => {
|
comments.map(async comment => {
|
||||||
const pull = await getPull(repo, comment.number);
|
const pull = await getPull(repo, comment.number);
|
||||||
if (pull.state !== 'open') return false;
|
|
||||||
|
if (pull.state !== 'open') return false; // filter out any closed pulls
|
||||||
|
if (PULL_LABEL_FILTER && !pull.labels.includes(PULL_LABEL_FILTER)) return false; // filter on label
|
||||||
|
|
||||||
return { comment, pull };
|
return { comment, pull };
|
||||||
})
|
})
|
||||||
)).filter(Boolean);
|
)).filter(Boolean);
|
||||||
@@ -38,8 +53,15 @@ export default async function() {
|
|||||||
})
|
})
|
||||||
)).filter(Boolean);
|
)).filter(Boolean);
|
||||||
|
|
||||||
console.log(records);
|
console.log(`Found ${records.length} outstanding failures`);
|
||||||
process.exit();
|
|
||||||
|
await Promise.all(
|
||||||
|
records.map(async record => {
|
||||||
|
console.log(`Re-testing PR #${record.pull.number}`);
|
||||||
|
const comment = await createComment(repo, records[0].pull.number, PULL_RETEST_BODY);
|
||||||
|
await deleteComment(repo, comment.id);
|
||||||
|
})
|
||||||
|
);
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
|
||||||
@@ -47,12 +69,12 @@ export default async function() {
|
|||||||
|
|
||||||
- [x] keep track of seen comment ids, only process new ones
|
- [x] keep track of seen comment ids, only process new ones
|
||||||
- [x] check the pr's status and only retest if no longer "Pending"
|
- [x] check the pr's status and only retest if no longer "Pending"
|
||||||
- [ ] add a retest comment
|
- [x] add a retest comment
|
||||||
- POST /repos/:owner/:repo/issues/:number/comments
|
- POST /repos/:owner/:repo/issues/:number/comments
|
||||||
|
- [x] delete the retest comment
|
||||||
- [ ] delete the build comment
|
- [ ] delete the build comment
|
||||||
- DELETE /repos/:owner/:repo/issues/comments/:comment_id
|
|
||||||
- [ ] delete ALL build failure comments
|
- [ ] delete ALL build failure comments
|
||||||
- [ ] delete the retest comment
|
- DELETE /repos/:owner/:repo/issues/comments/:comment_id
|
||||||
|
|
||||||
*/
|
*/
|
||||||
}
|
}
|
||||||
|
|||||||
7
src/lib/comments.mjs
Normal file
7
src/lib/comments.mjs
Normal file
@@ -0,0 +1,7 @@
|
|||||||
|
export async function createComment(repo, issueId, body) {
|
||||||
|
return repo.issues(issueId).comments.create({ body });
|
||||||
|
}
|
||||||
|
|
||||||
|
export async function deleteComment(repo, commentId) {
|
||||||
|
return repo.issues.comments(commentId).remove();
|
||||||
|
}
|
||||||
@@ -8,8 +8,11 @@ function usageError() {
|
|||||||
|
|
||||||
export default function(repo) {
|
export default function(repo) {
|
||||||
if (typeof repo !== 'string' || repo.length === 0) usageError();
|
if (typeof repo !== 'string' || repo.length === 0) usageError();
|
||||||
|
const settings = {
|
||||||
|
token: process.env.GITHUB_ACCESS_TOKEN,
|
||||||
|
};
|
||||||
|
|
||||||
const octo = new Octokat();
|
const octo = new Octokat(settings);
|
||||||
const [owner, name] = repo.split('/');
|
const [owner, name] = repo.split('/');
|
||||||
|
|
||||||
if (!owner || !name) usageError();
|
if (!owner || !name) usageError();
|
||||||
|
|||||||
@@ -1,7 +1,17 @@
|
|||||||
export default async function getEvents(repo, { body, actor } = {}) {
|
export default async function getEvents(repo, { body, actor } = {}) {
|
||||||
const { items: events } = await repo.events.fetch();
|
const events = await repo.events.fetch();
|
||||||
|
|
||||||
return events
|
const items = await (async () => {
|
||||||
|
// process 2 pages of events
|
||||||
|
if (events.nextPage) {
|
||||||
|
const moreEvents = await events.nextPage.fetch();
|
||||||
|
return [...events.items, ...moreEvents.items];
|
||||||
|
}
|
||||||
|
|
||||||
|
return events.items;
|
||||||
|
})();
|
||||||
|
|
||||||
|
return items
|
||||||
.map(comment => {
|
.map(comment => {
|
||||||
// only comments created by specific user
|
// only comments created by specific user
|
||||||
if (comment.type !== 'IssueCommentEvent') return false;
|
if (comment.type !== 'IssueCommentEvent') return false;
|
||||||
|
|||||||
Reference in New Issue
Block a user