Paths

Table of Contentst

Fix `flake8` unit test.
ClosedPublic
Actions

Authored by joshuaspence on May 6 2014, 1:52 AM.

Tags

None

Referenced Files

	F19803242: D8987.id21341.diff
	Sun, Mar 1, 9:55 PM

	F19792185: D8987.id21339.diff
	Thu, Feb 26, 8:59 AM

	F19716913: D8987.id21341.diff
	Thu, Feb 12, 12:36 PM

	F19421908: D8987.diff
	Dec 28 2025, 1:00 AM

	F18815388: D8987.id21341.diff
	Oct 21 2025, 1:59 AM

	F18749346: D8987.id21341.diff
	Oct 4 2025, 3:27 AM

	F18628256: D8987.diff
	Sep 16 2025, 5:58 AM

	F18419320: D8987.id21341.diff
	Aug 30 2025, 2:19 PM

View All Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

rARC213997dc9d6b: Fix `flake8` unit test.

Summary

This unit test is failing for me. It seems that I have a newer version of flake8. We should probably provide support for the most recent versions of any external tools, at least until we can implement version-specific stuff.

I am running 2.1.0 (pep8: 1.5.6, pyflakes: 0.8.1, mccabe: 0.2.1) CPython 2.7.6 on Linux.

Test Plan

arc unit.

Diff Detail

Repository

rARC Arcanist

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

joshuaspence updated this revision to Diff 21339.May 6 2014, 1:52 AM

joshuaspence retitled this revision from to Fix `flake8` unit test..

joshuaspence updated this object.

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

joshuaspence added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 6 2014, 1:52 AM

Herald added subscribers: Korvin, epriestley. · View Herald Transcript

Harbormaster completed remote builds in B259: Diff 21339.May 6 2014, 1:53 AM

Yep, agreed. This looks like it's a better result anyway (the message is more accurate).

This revision is now accepted and ready to land.May 6 2014, 1:59 AM

Closed by commit rARC213997dc9d6b (authored by @joshuaspence, committed by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

lint/

linter/

__tests__/

flake8/

undefined.lint-test

2 lines

Diff 21341

View Options

src/lint/linter/tests/flake8/undefined.lint-test

Fix `flake8` unit test.ClosedPublicActions