Page MenuHomeDevCentral
Differential D747

Support GitHub pull request event
ClosedPublic
Actions

Authored by dereckson on Dec 13 2016, 16:46.
Tags
None
Referenced Files
F4048924: D747.id1913.diff
Wed, Jan 22, 23:21
F4046389: D747.id1905.diff
Wed, Jan 22, 22:19
Unknown Object (File)
Sun, Jan 19, 21:19
Unknown Object (File)
Sun, Jan 19, 13:08
Unknown Object (File)
Sun, Jan 19, 04:47
Unknown Object (File)
Fri, Jan 17, 16:25
Unknown Object (File)
Mon, Jan 13, 08:47
Unknown Object (File)
Fri, Jan 10, 18:11
View All Files
Subscribers
alken-orin

Details

Reviewers
dereckson
Commits
rNOTIF50b0f14901f6: Support GitHub pull request event
Test Plan

Unit tests are provided.

Fire a real payload and ensure the expected notification is fired.

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Passed
Unit
Tests Passed
Branch
github-pull-request (branched from master)

Event Timeline

dereckson updated this revision to Diff 1874.Dec 13 2016, 16:46
dereckson retitled this revision from to WIP: Support GitHub pull request event.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: dereckson.
alken-orin added a subscriber: alken-orin.Dec 13 2016, 16:52

Build is green https://ci.nasqueron.org/job/test-notifications-phab/218/ for more details.

dereckson updated this revision to Diff 1904.Dec 19 2016, 21:20
dereckson edited edge metadata.

+assigned

alken-orin added a comment.Dec 19 2016, 21:26

Build is green https://ci.nasqueron.org/job/test-notifications-phab/221/ for more details.

dereckson updated this revision to Diff 1905.Dec 19 2016, 21:54

Virtually complete, but let's see if we can add labels.

alken-orin added a comment.Dec 19 2016, 21:58

Build is green https://ci.nasqueron.org/job/test-notifications-phab/222/ for more details.

dereckson accepted this revision.Dec 20 2016, 11:27
dereckson retitled this revision from WIP: Support GitHub pull request event to Support GitHub pull request event.
dereckson edited edge metadata.
In D747#11484, @dereckson wrote:

Virtually complete, but let's see if we can add labels.

Payload doesn't include any new/removed labels, only a ''labeled'' or ''unlabeled'' action, that's not helpful.

So we're merging this without labels, and we'll revisit labels later.

This revision is now accepted and ready to land.Dec 20 2016, 11:29
Closed by commit rNOTIF50b0f14901f6: Support GitHub pull request event (authored by dereckson, committed by dereckson). · Explain WhyDec 20 2016, 11:29
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
app/
Analyzers/
GitHub/
Events/
91 lines
resources/
lang/
en/
12 lines
tests/
Analyzers/
GitHub/
Events/
5 lines
50 lines
data/
payloads/
GitHubEvents/
470 lines

Diff 1905

View Options

app/Analyzers/GitHub/Events/PullRequestEvent.php

Loading...
View Options

resources/lang/en/GitHub.php

Loading...
View Options

tests/Analyzers/GitHub/Events/EventTest.php

Loading...
View Options

tests/Analyzers/GitHub/Events/PullRequestEventTest.php

Loading...
View Options

tests/data/payloads/GitHubEvents/pull_request.json

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator