.flake8 excludes moz.configure files:
Yet reviewbot is warning about flake8 errors in those files.
This is not related with the automated review process, the fact that is posted by the bot is just a side effect of the way how ./mach lint works for flake8 with wildcards. I think that this issue has been added once we've taken away the exclusion from flake8 in bug 1367092, more precisely here.
Since most of the work has been performed by ahal reassigning accordingly.
Good catch, yes this is a regression from that bug. Flake8 ignores the exclude directive completely if you pass in a direct path (as opposed to a directory), and this is what the reviewbot does via mach lint --outgoing.
mach lint --outgoing
Flake8 ignores the 'exclude' section of the .flake8.yml if you pass in a direct
path to a file. To get around this we have some custom logic to handle these
exclusions for us, but this custom logic didn't account for globs.