Branch: refs/heads/master
Home: https://github.com/jenkinsci/phabricator-plugin
Commit: 0a342c7b7ab054614063d1522cecfff372c4f08f
https://github.com/jenkinsci/phabricator-plugin/commit/0a342c7b7ab054614063d1522cecfff372c4f08f
Author: Aiden Scandella <[email protected]>
Date: 2015-08-10 (Mon, 10 Aug 2015)
Changed paths:
M phabricator-plugin.iml
M pom.xml
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/PhabricatorPlugin.java
A src/main/java/com/uber/jenkins/phabricator/conduit/ConduitAPIClient.java
A
src/main/java/com/uber/jenkins/phabricator/conduit/ConduitAPIException.java
M src/main/java/com/uber/jenkins/phabricator/conduit/Differential.java
M src/main/java/com/uber/jenkins/phabricator/conduit/DifferentialClient.java
A src/main/java/com/uber/jenkins/phabricator/tasks/ApplyPatchTask.java
M src/main/java/com/uber/jenkins/phabricator/tasks/PostCommentTask.java
A
src/test/java/com/uber/jenkins/phabricator/conduit/ConduitAPIClientTest.java
M
src/test/java/com/uber/jenkins/phabricator/conduit/DifferentialClientTest.java
A src/test/java/com/uber/jenkins/phabricator/tasks/ApplyPatchTaskTest.java
M src/test/java/com/uber/jenkins/phabricator/tasks/PostCommentTaskTest.java
M src/test/java/com/uber/jenkins/phabricator/utils/TestUtils.java
A
src/test/resources/com/uber/jenkins/phabricator/conduit/fetchDiffWithResponseArray.json
Log Message:
-----------
Call conduit over HTTP
This removes much of the reliance on `arc` being installed.
Also refactor apply patch into a task -- I realize this should be a separate
commit, but all of this stuff is intertwined and I wanted to break it out to
make this refactor easier.
Commit: 5f85f41f693f1f26ad34d8eb9fdfc27b13c8fe6d
https://github.com/jenkinsci/phabricator-plugin/commit/5f85f41f693f1f26ad34d8eb9fdfc27b13c8fe6d
Author: Aiden Scandella <[email protected]>
Date: 2015-08-10 (Mon, 10 Aug 2015)
Changed paths:
M phabricator-plugin.iml
M pom.xml
M src/main/java/com/uber/jenkins/phabricator/CommentBuilder.java
M
src/main/java/com/uber/jenkins/phabricator/ConduitCredentialsDescriptor.java
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
M
src/main/java/com/uber/jenkins/phabricator/conduit/ArcanistUsageException.java
M src/main/java/com/uber/jenkins/phabricator/conduit/ConduitAPIClient.java
M src/main/java/com/uber/jenkins/phabricator/conduit/Differential.java
M src/main/java/com/uber/jenkins/phabricator/conduit/DifferentialClient.java
M src/main/java/com/uber/jenkins/phabricator/tasks/ApplyPatchTask.java
M
src/main/java/com/uber/jenkins/phabricator/tasks/NonDifferentialBuildTask.java
M src/main/java/com/uber/jenkins/phabricator/tasks/PostCommentTask.java
M src/main/java/com/uber/jenkins/phabricator/tasks/Task.java
M src/main/java/com/uber/jenkins/phabricator/utils/Logger.java
M src/test/java/com/uber/jenkins/phabricator/BuildIntegrationTest.java
M src/test/java/com/uber/jenkins/phabricator/conduit/ArcanistClientTest.java
M
src/test/java/com/uber/jenkins/phabricator/conduit/ConduitAPIClientTest.java
M
src/test/java/com/uber/jenkins/phabricator/conduit/DifferentialClientTest.java
M src/test/java/com/uber/jenkins/phabricator/tasks/PostCommentTaskTest.java
M
src/test/resources/com/uber/jenkins/phabricator/conduit/fetchDiffResponseMissingDiff.json
M
src/test/resources/com/uber/jenkins/phabricator/conduit/fetchDiffWithResponseArray.json
M
src/test/resources/com/uber/jenkins/phabricator/conduit/validFetchDiffResponse.json
Log Message:
-----------
Cleanup
Commit: f3268e9153dcd2fa3cc55ca9d2f3c9e7dfb4317d
https://github.com/jenkinsci/phabricator-plugin/commit/f3268e9153dcd2fa3cc55ca9d2f3c9e7dfb4317d
Author: Aiden Scandella <[email protected]>
Date: 2015-08-10 (Mon, 10 Aug 2015)
Changed paths:
M src/main/java/com/uber/jenkins/phabricator/CommentBuilder.java
M src/main/java/com/uber/jenkins/phabricator/conduit/ArcanistClient.java
M src/test/java/com/uber/jenkins/phabricator/conduit/ArcanistClientTest.java
M src/test/java/com/uber/jenkins/phabricator/tasks/ApplyPatchTaskTest.java
Log Message:
-----------
Remove unused parseConduit
Commit: fd71a4fd994ff1307f5adbef5840f34e76f9f028
https://github.com/jenkinsci/phabricator-plugin/commit/fd71a4fd994ff1307f5adbef5840f34e76f9f028
Author: Aiden Scandella <[email protected]>
Date: 2015-08-10 (Mon, 10 Aug 2015)
Changed paths:
M src/main/java/com/uber/jenkins/phabricator/PhabricatorBuildWrapper.java
Log Message:
-----------
Address review feedback
Commit: dbe92a0f39cf236661770770c1dd075e5f610bc3
https://github.com/jenkinsci/phabricator-plugin/commit/dbe92a0f39cf236661770770c1dd075e5f610bc3
Author: Aiden Scandella <[email protected]>
Date: 2015-08-10 (Mon, 10 Aug 2015)
Changed paths:
M src/main/java/com/uber/jenkins/phabricator/conduit/DifferentialClient.java
Log Message:
-----------
Clarify conduit exception message
Commit: bf661b8eb6c49dfe4fb92ab98c1437cfd025af0f
https://github.com/jenkinsci/phabricator-plugin/commit/bf661b8eb6c49dfe4fb92ab98c1437cfd025af0f
Author: Aiden Scandella <[email protected]>
Date: 2015-08-10 (Mon, 10 Aug 2015)
Changed paths:
M src/main/java/com/uber/jenkins/phabricator/PhabricatorBuildWrapper.java
M src/main/java/com/uber/jenkins/phabricator/tasks/ApplyPatchTask.java
Log Message:
-----------
Address more review feedback
Commit: 27b43a1d8d55ccd80cc352ac8fd2549b237dc6fa
https://github.com/jenkinsci/phabricator-plugin/commit/27b43a1d8d55ccd80cc352ac8fd2549b237dc6fa
Author: Aiden Scandella <[email protected]>
Date: 2015-08-10 (Mon, 10 Aug 2015)
Changed paths:
M phabricator-plugin.iml
M pom.xml
M src/test/java/com/uber/jenkins/phabricator/utils/LoggerTest.java
Log Message:
-----------
Add test; remove trove4j
Commit: e881c467ccdaf30a6832f2e0a59677648594462c
https://github.com/jenkinsci/phabricator-plugin/commit/e881c467ccdaf30a6832f2e0a59677648594462c
Author: Aiden Scandella <[email protected]>
Date: 2015-08-10 (Mon, 10 Aug 2015)
Changed paths:
M src/test/java/com/uber/jenkins/phabricator/tasks/ApplyPatchTaskTest.java
Log Message:
-----------
Add another test for git failure
Commit: af1829efdecde01685f421214112c8eee35a4c03
https://github.com/jenkinsci/phabricator-plugin/commit/af1829efdecde01685f421214112c8eee35a4c03
Author: Aiden Scandella <[email protected]>
Date: 2015-08-11 (Tue, 11 Aug 2015)
Changed paths:
M phabricator-plugin.iml
M pom.xml
M src/main/java/com/uber/jenkins/phabricator/CommentBuilder.java
M
src/main/java/com/uber/jenkins/phabricator/ConduitCredentialsDescriptor.java
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/PhabricatorPlugin.java
M src/main/java/com/uber/jenkins/phabricator/conduit/ArcanistClient.java
M
src/main/java/com/uber/jenkins/phabricator/conduit/ArcanistUsageException.java
A src/main/java/com/uber/jenkins/phabricator/conduit/ConduitAPIClient.java
A
src/main/java/com/uber/jenkins/phabricator/conduit/ConduitAPIException.java
M src/main/java/com/uber/jenkins/phabricator/conduit/Differential.java
M src/main/java/com/uber/jenkins/phabricator/conduit/DifferentialClient.java
A src/main/java/com/uber/jenkins/phabricator/tasks/ApplyPatchTask.java
M
src/main/java/com/uber/jenkins/phabricator/tasks/NonDifferentialBuildTask.java
M src/main/java/com/uber/jenkins/phabricator/tasks/PostCommentTask.java
M src/main/java/com/uber/jenkins/phabricator/tasks/Task.java
M src/main/java/com/uber/jenkins/phabricator/utils/Logger.java
M src/test/java/com/uber/jenkins/phabricator/BuildIntegrationTest.java
M src/test/java/com/uber/jenkins/phabricator/conduit/ArcanistClientTest.java
A
src/test/java/com/uber/jenkins/phabricator/conduit/ConduitAPIClientTest.java
M
src/test/java/com/uber/jenkins/phabricator/conduit/DifferentialClientTest.java
A src/test/java/com/uber/jenkins/phabricator/tasks/ApplyPatchTaskTest.java
M src/test/java/com/uber/jenkins/phabricator/tasks/PostCommentTaskTest.java
M src/test/java/com/uber/jenkins/phabricator/utils/LoggerTest.java
M src/test/java/com/uber/jenkins/phabricator/utils/TestUtils.java
M
src/test/resources/com/uber/jenkins/phabricator/conduit/fetchDiffResponseMissingDiff.json
A
src/test/resources/com/uber/jenkins/phabricator/conduit/fetchDiffWithResponseArray.json
M
src/test/resources/com/uber/jenkins/phabricator/conduit/validFetchDiffResponse.json
Log Message:
-----------
Merge pull request #60 from uber/conduit-http
Call conduit over HTTP
Compare:
https://github.com/jenkinsci/phabricator-plugin/compare/a4d4d1998634...af1829efdecd
--
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 [email protected].
For more options, visit https://groups.google.com/d/optout.
