Differential D10358

Don't run "flake8" via stdin
ClosedPublic
Actions

Authored by epriestley on Aug 26 2014, 9:31 PM.

Details

Reviewers

btrahan

Commits

rARCe4bbcaf8f7bf: Don't run "flake8" via stdin

Summary

flake8 v2.2.3 (released Aug 25) broke this. Just drop the stdin stuff.

Test Plan

User confirmed in IRC that this resolves the issue.

Diff Detail

Repository

rARC Arcanist

Branch

flake8

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 2356
Build 2360: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 24933.Aug 26 2014, 9:31 PM

epriestley retitled this revision from to Don't run "flake8" via stdin.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

Herald added a subscriber: epriestley. · View Herald TranscriptAug 26 2014, 9:31 PM

Harbormaster completed remote builds in B2356: Diff 24933.Aug 26 2014, 9:32 PM

btrahan accepted this revision.Aug 26 2014, 9:33 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Aug 26 2014, 9:33 PM

Closed by commit rARCe4bbcaf8f7bf (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

lint/

linter/

ArcanistFlake8Linter.php

8 lines

Diff 24933

View Options

src/lint/linter/ArcanistFlake8Linter.php

Don't run "flake8" via stdinClosedPublicActions