Page MenuHomeDevCentral
Differential D284

Nasqueron\Notifications\Analyzers\Phabricator namespace
ClosedPublic
Actions

Authored by dereckson on Feb 18 2016, 00:12.
Tags
None
Referenced Files
F45722195: D284.id669.diff
Sun, Aug 23, 07:26
F45721252: D284.id670.diff
Sun, Aug 23, 07:21
F45692003: D284.diff
Sun, Aug 23, 04:38
F45672823: D284.id669.diff
Sun, Aug 23, 02:43
Unknown Object (File)
Thu, Aug 20, 08:44
Unknown Object (File)
Tue, Aug 18, 03:35
Unknown Object (File)
Mon, Aug 17, 15:04
Unknown Object (File)
Mon, Aug 17, 14:00
View All Files
Subscribers
alken-orin

Details

Reviewers
dereckson
Commits
rNOTIF77b457cdb9f6: Nasqueron\Notifications\Analyzers\Phabricator namespace
Summary
  • Moved Phabricatro analyzer classes to a new Phabricator namespace.
  • Renamed PhabricatorGroupMapping class to ProjectGroupMapping, to be coherent with the RepositoryGroupMapping class.
Test Plan

Fire a Phabricator payload to the notifications center.

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Passed
Unit
No Test Coverage
Branch
test/PhabricatorGroupMapping

Event Timeline

dereckson updated this revision to Diff 669.Feb 18 2016, 00:12
dereckson retitled this revision from to Nasqueron\Notifications\Analyzers\Phabricator namespace.
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.Feb 18 2016, 00:12

Build started: https://ci.nasqueron.org/job/test-notifications-phab/79/ (console: https://ci.nasqueron.org/job/test-notifications-phab/79/console)

alken-orin added a comment.Feb 18 2016, 00:20

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

dereckson accepted this revision.Feb 18 2016, 00:23
dereckson updated this object.
dereckson edited edge metadata.
This revision is now accepted and ready to land.Feb 18 2016, 00:23
Closed by commit rNOTIF77b457cdb9f6: Nasqueron\Notifications\Analyzers\Phabricator namespace (authored by dereckson, committed by dereckson). · Explain WhyFeb 18 2016, 00:24
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 669

View Options

app/Analyzers/Phabricator/PhabricatorPayloadAnalyzer.php

Loading...
View Options

app/Analyzers/Phabricator/PhabricatorPayloadAnalyzerConfiguration.php

Loading...
View Options

app/Analyzers/Phabricator/ProjectGroupMapping.php

Loading...
View Options

app/Analyzers/PhabricatorGroupMapping.php

Loading...
View Options

app/Analyzers/PhabricatorPayloadAnalyzer.php

Loading...
View Options

app/Analyzers/PhabricatorPayloadAnalyzerConfiguration.php

Loading...
View Options

app/Notifications/PhabricatorNotification.php

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