Branch: refs/heads/master 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. Commit: 3b367665d5de002bb9d188d25597b57465344faa https://github.com/jenkinsci/phabricator-plugin/commit/3b367665d5de002bb9d188d25597b57465344faa Author: Aiden Scandella <s...@ndella.com> Date: 2015-06-07 (Sun, 07 Jun 2015) Changed paths: M src/main/java/com/uber/jenkins/phabricator/conduit/Differential.java Log Message: ----------- Merge URLs properly for missing slashes References #27 Commit: 8d89284d9977b39517d41ca8fcfb4ecde1b3e424 https://github.com/jenkinsci/phabricator-plugin/commit/8d89284d9977b39517d41ca8fcfb4ecde1b3e424 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: ----------- Merge pull request #28 from uber/handle-errors Raise arcanist usage exceptions on errors Commit: 498c7fb28375184d2168ecd05ea2b9b6236e0f6c https://github.com/jenkinsci/phabricator-plugin/commit/498c7fb28375184d2168ecd05ea2b9b6236e0f6c Author: Aiden Scandella <s...@ndella.com> Date: 2015-06-07 (Sun, 07 Jun 2015) Changed paths: M src/main/java/com/uber/jenkins/phabricator/conduit/Differential.java Log Message: ----------- Merge pull request #29 from uber/handle-url Merge URLs properly for missing slashes Commit: fb52f006bda7969450262aefc623ebaa7e13b446 https://github.com/jenkinsci/phabricator-plugin/commit/fb52f006bda7969450262aefc623ebaa7e13b446 Author: Aiden Scandella <s...@ndella.com> Date: 2015-06-07 (Sun, 07 Jun 2015) Changed paths: M pom.xml Log Message: ----------- [maven-release-plugin] prepare release phabricator-plugin-1.4 Compare: https://github.com/jenkinsci/phabricator-plugin/compare/9fd0f8736458...fb52f006bda7 -- 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.