Page MenuHomePhabricator
Differential D15438

Add a GitHub API Future
ClosedPublic
Actions

Authored by epriestley on Mar 8 2016, 4:29 PM.
Tags
None
Referenced Files
F14001978: D15438.id37208.diff
Fri, Oct 25, 1:49 PM
F13976205: D15438.diff
Fri, Oct 18, 1:15 PM
F13963364: D15438.id37204.diff
Tue, Oct 15, 3:27 PM
Unknown Object (File)
Sun, Oct 13, 10:56 PM
Unknown Object (File)
Sun, Oct 13, 8:27 PM
Unknown Object (File)
Oct 4 2024, 2:17 AM
Unknown Object (File)
Sep 23 2024, 9:24 PM
Unknown Object (File)
Sep 17 2024, 5:43 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T10538: Bridge GitHub to Phabricator
Commits
rPHU7e2eca1f76d1: Add a GitHub API Future
Summary

Ref T10538. See next change for usage.

Test Plan

See next change.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 37204.Mar 8 2016, 4:29 PM
epriestley retitled this revision from to Add a GitHub API Future.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T10538: Bridge GitHub to Phabricator.
chad accepted this revision.Mar 8 2016, 5:10 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Mar 8 2016, 5:10 PM
epriestley updated this revision to Diff 37207.Mar 8 2016, 6:28 PM
epriestley edited edge metadata.
  • Minor fixes for rate limit / not modified detection.
Closed by commit rPHU7e2eca1f76d1: Add a GitHub API Future (authored by epriestley, committed by epriestley). · Explain WhyMar 8 2016, 6:29 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
4 lines
future/
github/
130 lines
49 lines

Diff 37208

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/future/github/PhutilGitHubFuture.php

Loading...
View Options

src/future/github/PhutilGitHubResponse.php

Loading...
Phabricator · Built by Phacility