{"push_id":656784977,"size":13,"distinct_size":13,"ref":"refs/heads/master","head":"73aca1f9f0b1984707b43579b2275d3179217875","before":"977baacc329ce690d4e5c0304f4c50e352e7644f","commits":[{"sha":"a4d33ef117aa8702181154b60ce1ce52bcfc119b","author":{"email":"c028c213ed5efcf30c3f4fc7361dbde0c893c5b7@joshuaspence.com","name":"Joshua Spence"},"message":"Use phutil_json_decode instead of json_decode\n\nSummary: Generally, `phutil_json_decode` should be preferred over `json_decode`.\n\nTest Plan: Eyeballed.\n\nReviewers: epriestley, #blessed_reviewers\n\nReviewed By: epriestley, #blessed_reviewers\n\nSubscribers: aurelijus, Korvin, epriestley\n\nDifferential Revision: https://secure.phabricator.com/D12678","distinct":true,"url":"https://api.github.com/repos/haskell-infra/arcanist/commits/a4d33ef117aa8702181154b60ce1ce52bcfc119b"},{"sha":"f477682726ba4779b6e7b264ad665e29dbf206fc","author":{"email":"c028c213ed5efcf30c3f4fc7361dbde0c893c5b7@joshuaspence.com","name":"Joshua Spence"},"message":"Fix a typo\n\nSummary: This variable is misnamed.\n\nTest Plan: `arc unit`\n\nReviewers: epriestley, #blessed_reviewers\n\nReviewed By: epriestley, #blessed_reviewers\n\nSubscribers: Korvin, epriestley\n\nDifferential Revision: https://secure.phabricator.com/D12695","distinct":true,"url":"https://api.github.com/repos/haskell-infra/arcanist/commits/f477682726ba4779b6e7b264ad665e29dbf206fc"},{"sha":"2b6568a4b983fb81e388a1611acf959839a087cd","author":{"email":"c028c213ed5efcf30c3f4fc7361dbde0c893c5b7@joshuaspence.com","name":"Joshua Spence"},"message":"Remove pre-commit hooks\n\nSummary: Ref T7674. The `arc git-hook-pre-receive` and `arc svn-hook-pre-commit` workflows are being removed.\n\nTest Plan: N/A\n\nReviewers: epriestley, #blessed_reviewers\n\nReviewed By: epriestley, #blessed_reviewers\n\nSubscribers: Korvin, epriestley\n\nManiphest Tasks: T7674\n\nDifferential Revision: https://secure.phabricator.com/D12690","distinct":true,"url":"https://api.github.com/repos/haskell-infra/arcanist/commits/2b6568a4b983fb81e388a1611acf959839a087cd"},{"sha":"0a711315f86ba6a39a48776b8b3ba65591967902","author":{"email":"c028c213ed5efcf30c3f4fc7361dbde0c893c5b7@joshuaspence.com","name":"Joshua Spence"},"message":"Fix a failing XML unit test\n\nSummary: Ref T7215. This test case is consistently failing locally (with a `LIBXML_VERSION` of `20902`).\n\nTest Plan: `arc unit`\n\nReviewers: epriestley, #blessed_reviewers\n\nReviewed By: epriestley, #blessed_reviewers\n\nSubscribers: Korvin, epriestley\n\nManiphest Tasks: T7215\n\nDifferential Revision: https://secure.phabricator.com/D12706","distinct":true,"url":"https://api.github.com/repos/haskell-infra/arcanist/commits/0a711315f86ba6a39a48776b8b3ba65591967902"},{"sha":"4330b27c07d2e88de75295bc818e0386b35bcab4","author":{"email":"c028c213ed5efcf30c3f4fc7361dbde0c893c5b7@joshuaspence.com","name":"Joshua Spence"},"message":"Remove support for passing data via STDIN\n\nSummary: Ref T7674. Remove support for passing data to linters via `STDIN`. This functionality exists primarily for pre-receive workflows (which don't have a working copy to act on), but these workflows are going away soon.\n\nTest Plan: `arc unit`\n\nReviewers: #blessed_reviewers, epriestley\n\nReviewed By: #blessed_reviewers, epriestley\n\nSubscribers: Korvin, epriestley\n\nManiphest Tasks: T7674\n\nDifferential Revision: https://secure.phabricator.com/D12696","distinct":true,"url":"https://api.github.com/repos/haskell-infra/arcanist/commits/4330b27c07d2e88de75295bc818e0386b35bcab4"},{"sha":"0846c6aff5c4da7b0c3a9f959bf77368329c5775","author":{"email":"c028c213ed5efcf30c3f4fc7361dbde0c893c5b7@joshuaspence.com","name":"Joshua Spence"},"message":"Remove commit linter\n\nSummary: Ref T7674. This linter doesn't make sense without commit hooks.\n\nTest Plan: N/A\n\nReviewers: epriestley, #blessed_reviewers\n\nReviewed By: epriestley, #blessed_reviewers\n\nSubscribers: Korvin, epriestley\n\nManiphest Tasks: T7674\n\nDifferential Revision: https://secure.phabricator.com/D12697","distinct":true,"url":"https://api.github.com/repos/haskell-infra/arcanist/commits/0846c6aff5c4da7b0c3a9f959bf77368329c5775"},{"sha":"c00899ad60ff9ac96746839a7abef65c337e2359","author":{"email":"bc4bd52f99de2e3a917136da4aaf9cc42f1711b5@gmail.com","name":"Aviv Eyal"},"message":"Modernize Script-and-Regex linter for ConfigurationDriven\n\nSummary: Support and prefer configuration from .arclint over Configuration Manager\n\nTest Plan: arc lint with several combinations of values in .arcconfig and .arclint.\n\nReviewers: joshuaspence, epriestley, #blessed_reviewers\n\nReviewed By: joshuaspence, epriestley, #blessed_reviewers\n\nSubscribers: vrusinov, jpoehls, Korvin, epriestley\n\nDifferential Revision: https://secure.phabricator.com/D10704","distinct":true,"url":"https://api.github.com/repos/haskell-infra/arcanist/commits/c00899ad60ff9ac96746839a7abef65c337e2359"},{"sha":"8919a9c5b55ec1802b20c4bb06f3a97253707b2c","author":{"email":"c028c213ed5efcf30c3f4fc7361dbde0c893c5b7@joshuaspence.com","name":"Joshua Spence"},"message":"Remove hook functionality\n\nSummary: Fixes T7674. Remove remaining commit hook functionality.\n\nTest Plan: Unit tests still pass?\n\nReviewers: #blessed_reviewers, epriestley\n\nReviewed By: #blessed_reviewers, epriestley\n\nSubscribers: Korvin, epriestley\n\nManiphest Tasks: T7674\n\nDifferential Revision: https://secure.phabricator.com/D12698","distinct":true,"url":"https://api.github.com/repos/haskell-infra/arcanist/commits/8919a9c5b55ec1802b20c4bb06f3a97253707b2c"},{"sha":"111b9b035aecdfe2ddc8d0cf83b98c9829204427","author":{"email":"46f1a0bd5592a2f9244ca321b129902a06b53e03@epriestley.com","name":"epriestley"},"message":"Add a --conduit-token parameter to `arc`\n\nSummary: Ref T5955. This makes it easier to write scripts which call Conduit via `arc call-conduit`.\n\nTest Plan: Used `arc --conduit-token ... call-conduit user.whoami` to make calls as various users.\n\nReviewers: btrahan\n\nReviewed By: btrahan\n\nSubscribers: mbishopim3, epriestley\n\nManiphest Tasks: T5955\n\nDifferential Revision: https://secure.phabricator.com/D12717","distinct":true,"url":"https://api.github.com/repos/haskell-infra/arcanist/commits/111b9b035aecdfe2ddc8d0cf83b98c9829204427"},{"sha":"6f7bedaceb6857a084b4914679a43fdedfb720da","author":{"email":"bc4bd52f99de2e3a917136da4aaf9cc42f1711b5@gmail.com","name":"Aviv Eyal"},"message":"Fix linter config check\n\nSummary: Fixes T6311 and T5124 by returning all configured linters from `buildLinter()`, and making `ArcanistExternalLinter::checkBinaryConfiguration()` not crash if there's no executable to run.\n\nTest Plan: `arc linters` in rP shows \"Configured\" and \"ERROR\" as appropriate; Adding a broken linter to `.arclint` in rARC doesn't invoke it's not actually needed, and prints error if it is.\n\nReviewers: joshuaspence, epriestley, #blessed_reviewers\n\nReviewed By: joshuaspence, epriestley, #blessed_reviewers\n\nSubscribers: Korvin, epriestley\n\nManiphest Tasks: T6311\n\nDifferential Revision: https://secure.phabricator.com/D10773","distinct":true,"url":"https://api.github.com/repos/haskell-infra/arcanist/commits/6f7bedaceb6857a084b4914679a43fdedfb720da"},{"sha":"3e36b3fc430d227d6bd7d478bb4713c567e5086c","author":{"email":"18205a07c9a4ae397555c5a2c68a8767aef4ba0c@dkd.de","name":"Sascha Egerer"},"message":"Fix broken ArcanistPhpunitTestResultParser\n\nSummary:\nThe ArcanistPhpunitTestResultParser has been changed\nto use phutil_json_decode instead of json_decode in\nrARCa4d33ef117aa8702181154b60ce1ce52bcfc119b\nThat has broken the functionality as json_decode has returned\nan object before but phutil_json_decode does return an array.\nThe code has now been adopted to work with the array result\ninstead of an object.\n\nTest Plan: Run a phpunit test case\n\nReviewers: #blessed_reviewers, epriestley\n\nReviewed By: #blessed_reviewers, epriestley\n\nSubscribers: aurelijus, epriestley\n\nDifferential Revision: https://secure.phabricator.com/D12751","distinct":true,"url":"https://api.github.com/repos/haskell-infra/arcanist/commits/3e36b3fc430d227d6bd7d478bb4713c567e5086c"},{"sha":"6c5d12d839930a494c9a760699f8530b9cf5e235","author":{"email":"46f1a0bd5592a2f9244ca321b129902a06b53e03@epriestley.com","name":"epriestley"},"message":"Make --conduit-token work without requiring .arcrc\n\nSummary:\nRef T5955. This logic is a little cleaner than the previous version.\n\nDon't require `~/.arcrc` to exist if the caller provides `--conduit-token`.\n\nTest Plan:\n  - Made calls with `--conduit-token` and no `~/.arcrc`.\n  - Made calls with `--conduit-token` and a normal `~/.arcrc`.\n  - Made calls with normal `arc`.\n\nReviewers: btrahan\n\nReviewed By: btrahan\n\nSubscribers: mbishopim3, epriestley\n\nManiphest Tasks: T5955\n\nDifferential Revision: https://secure.phabricator.com/D12750","distinct":true,"url":"https://api.github.com/repos/haskell-infra/arcanist/commits/6c5d12d839930a494c9a760699f8530b9cf5e235"},{"sha":"73aca1f9f0b1984707b43579b2275d3179217875","author":{"email":"46f1a0bd5592a2f9244ca321b129902a06b53e03@epriestley.com","name":"epriestley"},"message":"Allow Arcanist to parse bare revision IDs from \"Differential Revisions:\" field\n\nSummary: Ref T8087. Prepares for eventually making these optional after T6030. See also T7443.\n\nTest Plan:\n  - See the next change for the server-side part of this.\n  - With both patches applied, rigged the server to return `D123`.\n  - Created a revision, saw bare `D123`.\n  - Updated it with bare `D123`, things worked properly.\n  - Created this revision with full URIs.\n\nReviewers: btrahan\n\nReviewed By: btrahan\n\nSubscribers: cburroughs, epriestley\n\nManiphest Tasks: T8087\n\nDifferential Revision: https://secure.phabricator.com/D12748","distinct":true,"url":"https://api.github.com/repos/haskell-infra/arcanist/commits/73aca1f9f0b1984707b43579b2275d3179217875"}]}