Don't run "flake8" via stdin
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F15508290: D10358.diff
	Wed, Apr 16, 4:12 AM

	F15499514: D10358.id.diff
	Sun, Apr 13, 2:58 PM

	F15499353: D10358.diff
	Sun, Apr 13, 2:50 PM

	F15468765: D10358.diff
	Fri, Apr 4, 5:28 AM

	F15431757: D10358.id24933.diff
	Mar 24 2025, 2:17 PM

	F15404247: D10358.id24933.diff
	Mar 18 2025, 6:55 AM

	F15404245: D10358.id24936.diff
	Mar 18 2025, 6:55 AM

	Unknown Object (File)
	Feb 19 2025, 10:07 PM

View All Files

Subscribers

epriestley

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

Lint

Lint Skipped

Unit

Tests Skipped

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 24936

View Options

src/lint/linter/ArcanistFlake8Linter.php

Don't run "flake8" via stdinClosedPublicActions