Page MenuHomePhabricator
Differential D9489

Tweak error and status messages for commit ranges
ClosedPublic
Actions

Authored by epriestley on Jun 11 2014, 10:30 PM.
Tags
None
Referenced Files
F13063289: D9489.diff
Sat, Apr 20, 6:51 AM
Unknown Object (File)
Wed, Apr 3, 10:10 AM
Unknown Object (File)
Fri, Mar 29, 4:13 PM
Unknown Object (File)
Mar 14 2024, 6:03 AM
Unknown Object (File)
Mar 1 2024, 3:12 AM
Unknown Object (File)
Feb 27 2024, 11:14 PM
Unknown Object (File)
Feb 8 2024, 5:24 PM
Unknown Object (File)
Feb 4 2024, 4:04 AM
View All Files
Subscribers
epriestley

Details

Reviewers
talshiri
Commits
rARC9492b4ecba12: Tweak error and status messages for commit ranges
Summary

Improve/clarify some error messages a bit, hopefully.

Test Plan

Ran arc which, arc diff, etc., with various explicit, implicit, and --head flags. Read error messages, didn't catch anything too awkward.

Diff Detail

Repository
rARC Arcanist
Branch
messages
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1018
Build 1018: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 22723.Jun 11 2014, 10:30 PM
epriestley retitled this revision from to Tweak error and status messages for commit ranges.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: talshiri.
Herald added a subscriber: epriestley. · View Herald TranscriptJun 11 2014, 10:30 PM
Harbormaster completed remote builds in B1018: Diff 22723.Jun 11 2014, 10:31 PM
epriestley updated this revision to Diff 22725.Jun 11 2014, 10:32 PM
  • Use %R ("Readable Argument") in the Mercurial construction.
Harbormaster completed remote builds in B1019: Diff 22725.Jun 11 2014, 10:33 PM
talshiri added inline comments.Jun 11 2014, 10:52 PM
src/repository/api/ArcanistGitAPI.php
132

getBaseCommit() returns a SHA?

149–153

This is exactly infinite time better than my phrasing

epriestley added inline comments.Jun 11 2014, 10:56 PM
src/repository/api/ArcanistGitAPI.php
132

It may or may not return a SHA. If invoked as arc diff X, it will return a SHA, but if a --base rule is used to figure it out it may not.

talshiri accepted this revision.Jun 11 2014, 10:58 PM
talshiri edited edge metadata.
talshiri added inline comments.
src/repository/api/ArcanistGitAPI.php
132

oic

This revision is now accepted and ready to land.Jun 11 2014, 10:58 PM
epriestley closed this revision.Jun 11 2014, 11:35 PM
epriestley updated this revision to Diff 22746.

Closed by commit rARC9492b4ecba12 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
repository/
api/
76 lines
workflow/
21 lines
47 lines

Diff 22723

View Options

src/repository/api/ArcanistGitAPI.php

Loading...
View Options

src/workflow/ArcanistDiffWorkflow.php

Loading...
View Options

src/workflow/ArcanistWhichWorkflow.php

Loading...
Phabricator · Built by Phacility