HomePhorge

Add two new conduit methods: createlintresults and getdiffproperty

Description

Add two new conduit methods: createlintresults and getdiffproperty

Summary:
We need createlintresults because we are doing extended
static analysis offline, and thus we need to be able to update the
lint results associated with a diff. This is similar to
updateunitresults, but "create" is more accurate than "update" since
we never need to change existing lint results.

getdiffproperty is used by the client to ensure it isn't creating any
duplicates lint results. It's the symmetric operation to
setdiffproperty, which already exists.

Test Plan:
We have a new offline linter that I used to test. This
linter calls getdiffproperty on every run.

  1. Tested updating an existing set of lint results by first running

"arc diff" with lint errors caught by the local linter, then later
running offline analysis which catches one other error and updates via
createlintresults. Ensured the differential lint results were as
expected.

  1. Tested the creation of an entirely new diff property through

createlintresults. I first ran "arc diff --nolint" to skip all lint
results, then ran offline analysis which caught an error and updated
through createlintresults. Ensured differential lint results were as
expected.

Reviewers: epriestley

Reviewed By: epriestley

CC: dpepper, aran, mgummelt, jungejason, epriestley

Differential Revision: 868

Details

Provenance
mgummeltAuthored on Aug 23 2011, 5:16 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP39ca1e7e3904: Add user.info and phid.info Conduit methods
Branches
Unknown
Tags
Unknown

Event Timeline