Fix some bad three-state logic in thrift_linter.

Review Request #3621 - Created March 28, 2016 and submitted

Information
Benjy Weinberger
pants
Reviewers
pants-reviews
nhoward_tw

It was relying on a default of None in a boolean option
to detect if the option was explicitly provided or not.
But we never want boolean options to be None, as three
state logic is pointlessly complicated (and a pending
change enforces this).

This change replaces that logic with a proper check.

CI passed here: https://travis-ci.org/pantsbuild/pants/builds/118877858

Benjy Weinberger
Benjy Weinberger
Benjy Weinberger
Nick Howard (Twitter)
Stu Hood
Nick Howard (Twitter)
Benjy Weinberger
Review request changed

Status: Closed (submitted)

Change Summary:

5a7ff93edc0c8307d3d31d71061eb5ea5a43da36

Loading...