Homec4science

Add two new conduit methods: createlintresults and getdiffproperty

Authored by mgummelt <mgummelt@fb.com> on Aug 24 2011, 02:16.

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

Committed
mgummelt <mgummelt@fb.com>Aug 31 2011, 00:14
Pushed
aubortJan 31 2017, 17:16
Parents
rPH39ca1e7e3904: Add user.info and phid.info Conduit methods
Branches
Unknown
Tags
Unknown

Event Timeline

mgummelt <mgummelt@fb.com> committed rPHee6c24b98d1a: Add two new conduit methods: createlintresults and getdiffproperty (authored by mgummelt <mgummelt@fb.com>).Aug 31 2011, 00:14