Page MenuHomePhabricator
Differential D9797

Refactor author handling in the Mercurial API
ClosedPublic
Actions

Authored by richardvanvelzen on Jul 2 2014, 3:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 18, 8:46 PM
Unknown Object (File)
Thu, Nov 14, 5:41 AM
Unknown Object (File)
Sun, Nov 10, 7:10 AM
Unknown Object (File)
Sun, Nov 10, 6:25 AM
Unknown Object (File)
Thu, Nov 7, 4:22 AM
Unknown Object (File)
Tue, Nov 5, 5:14 AM
Unknown Object (File)
Sat, Nov 2, 6:39 AM
Unknown Object (File)
Tue, Oct 29, 6:52 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rARC792204db95cc: Refactor author handling in the Mercurial API
Required Signatures
L28 Phacility Individual Contributor License Agreement
Summary

There were two callsites which needed some information from the username. Only one worked "correctly", causing arc diff to not amend commits anymore because the author could not be parser.

Test Plan

run arc diff with changes

Diff Detail

Repository
rARC Arcanist
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

richardvanvelzen updated this revision to Diff 23514.Jul 2 2014, 3:06 PM
richardvanvelzen retitled this revision from to Refactor author handling in the Mercurial API.
richardvanvelzen updated this object.
richardvanvelzen edited the test plan for this revision. (Show Details)
richardvanvelzen added a reviewer: epriestley.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJul 2 2014, 3:06 PM
Herald added a required legal document: L28 Phacility Individual Contributor License Agreement. · View Herald Transcript
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
Harbormaster completed remote builds in B1484: Diff 23514.Jul 2 2014, 3:07 PM
richardvanvelzen updated this revision to Diff 23515.Jul 2 2014, 3:07 PM
richardvanvelzen edited edge metadata.

Debug-less

Harbormaster completed remote builds in B1485: Diff 23515.Jul 2 2014, 3:08 PM
epriestley accepted this revision.Jul 2 2014, 8:58 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jul 2 2014, 8:58 PM
epriestley closed this revision.Jul 2 2014, 8:59 PM
epriestley updated this revision to Diff 23520.

Closed by commit rARC792204db95cc (authored by Richard van Velzen <rvanvelzen@experty.com>, committed by @epriestley).

Revision Contents
Changeset List

PathSize
src/
repository/
api/
36 lines

Diff 23520

View Options

src/repository/api/ArcanistMercurialAPI.php

Loading...
Phabricator · Built by Phacility