Configure scripts don't pass flake8

RESOLVED FIXED

Status

defect
RESOLVED FIXED
Last year
5 months ago

People

(Reporter: ahal, Assigned: sylvestre)

Tracking

(Blocks 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Bug 1406668 attempted to enable flake8 on build/moz.configure/*.configure files, however due to a bug in our flake8 integration those files were not actually being linted unless you explicitly specified the file path.

The flake8 integration issue was since fixed by bug 1473308, but I had to skip most of the *.configure files (they were either never flake8 compliant, or bustage slipped in after the fact).

To re-enable them, remove this line:
https://searchfox.org/mozilla-central/rev/bdfd20ef30d521b57d5b6feeda71325e8b4cad66/.flake8#8

Note there other *.configure scripts scattered around the tree that are skipped in there as well.
Blocks: flake8
Sigh, does that mean we'll have even more rules to deal with?  The rules for whitespace lines already make writing *.configure files annoying.
build/moz.configure/*.configure is almost green and almost didn't regress in 10 months.
What are your concerns? (these checks are enabled at review phase btw)
Keeping it open for the other configure files
Assignee: nobody → sledru
Keywords: leave-open
Comment on attachment 8994829 [details]
bug 1478338 - Fix some flake8 regression in moz.configure/ r?froydnj

Nathan Froyd [:froydnj] has approved the revision.

https://phabricator.services.mozilla.com/D2356
Attachment #8994829 - Flags: review+
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4738f8111a82
Fix some flake8 regression in moz.configure/ r=froydnj
I will have a look, thanks!
Flags: needinfo?(sledru)

The leave-open keyword is there and there is no activity for 6 months.
:sylvestre, maybe it's time to close this bug?

Flags: needinfo?(sledru)

Closing, we should open a new bug to continue the work

Status: NEW → RESOLVED
Closed: 5 months ago
Flags: needinfo?(sledru)
Keywords: leave-open
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.