Branch: refs/heads/handle-errors
Home: https://github.com/jenkinsci/phabricator-plugin
Commit: 7025888bd449f671d0ddef100dc459b5b083126f
https://github.com/jenkinsci/phabricator-plugin/commit/7025888bd449f671d0ddef100dc459b5b083126f
Author: Aiden Scandella <s...@ndella.com>
Date: 2015-06-07 (Sun, 07 Jun 2015)
Changed paths:
M src/main/java/com/uber/jenkins/phabricator/PhabricatorBuildWrapper.java
M src/main/java/com/uber/jenkins/phabricator/PhabricatorNotifier.java
M src/main/java/com/uber/jenkins/phabricator/conduit/ArcanistClient.java
A
src/main/java/com/uber/jenkins/phabricator/conduit/ArcanistUsageException.java
M src/main/java/com/uber/jenkins/phabricator/conduit/Differential.java
Log Message:
-----------
Raise arcanist usage exceptions on errors
Problems like #19 are misleading because it looks like a JSON parsing
error, when we know there is no JSON to parse. Make the callers deal
with the errors and print a useful error message as to why things are
failing.
--
You received this message because you are subscribed to the Google Groups
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to jenkinsci-commits+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
