Don't use `flake8: noqa`.

This suppresses all errors in the file.  Use `noqa`, although pep8
doesn't honor this for all errors.

There may be a plugin which helps.

7 files changed, 9 insertions, 10 deletions

diff --git a/src/mailman/commands/cli_control.py b/src/mailman/commands/cli_control.py
index 6a90a510d..a99a62dbc 100644
--- a/src/mailman/commands/cli_control.py
+++ b/src/mailman/commands/cli_control.py
@@ -93,7 +93,7 @@ class Start:
                 self.parser.error(
                     _('A previous run of GNU Mailman did not exit '
                       'cleanly.  Try using --force.'))
-        def log(message):                           # flake8: noqa
+        def log(message):                           # noqa
             if not args.quiet:
                 print(message)
         # Try to find the path to a valid, existing configuration file, and
diff --git a/src/mailman/commands/cli_lists.py b/src/mailman/commands/cli_lists.py
index 8720f91c5..0b077543e 100644
--- a/src/mailman/commands/cli_lists.py
+++ b/src/mailman/commands/cli_lists.py
@@ -89,7 +89,7 @@ class Lists:
             if not args.quiet:
                 print(_('No matching mailing lists found'))
             return
-        count = len(mailing_lists)                  # flake8: noqa
+        count = len(mailing_lists)                  # noqa
         if not args.quiet:
             print(_('$count matching mailing lists found:'))
         # Calculate the longest identifier.
@@ -198,8 +198,7 @@ class Create:
             invalid_owners = [owner for owner in args.owners
                               if not validator.is_valid(owner)]
             if invalid_owners:
-                invalid = COMMASPACE.join(          # flake8: noqa
-                    sorted(invalid_owners))
+                invalid = COMMASPACE.join(sorted(invalid_owners))   # noqa
                 self.parser.error(_('Illegal owner addresses: $invalid'))
                 return
         try:
diff --git a/src/mailman/commands/cli_qfile.py b/src/mailman/commands/cli_qfile.py
index c66ed85d7..0ed078ff3 100644
--- a/src/mailman/commands/cli_qfile.py
+++ b/src/mailman/commands/cli_qfile.py
@@ -80,7 +80,7 @@ class QFile:
                 else:
                     printer.pprint(obj)
             print(_('[----- end pickle -----]'))
-        count = len(m)                              # flake8: noqa
+        count = len(m)                              # noqa
         banner = _("The variable 'm' contains $count objects")
         if args.interactive:
             interact(banner=banner)
diff --git a/src/mailman/commands/cli_status.py b/src/mailman/commands/cli_status.py
index 7cc69850a..dae9cfe0a 100644
--- a/src/mailman/commands/cli_status.py
+++ b/src/mailman/commands/cli_status.py
@@ -50,7 +50,7 @@ class Status:
             message = _('GNU Mailman is stopped (stale pid: $pid)')
         else:
             hostname, pid, tempfile = lock.details
-            fqdn_name = socket.getfqdn()            # flake8: noqa
+            fqdn_name = socket.getfqdn()            # noqa
             assert status is WatcherState.host_mismatch, (
                 'Invalid enum value: %s' % status)
             message = _('GNU Mailman is in an unexpected state '
diff --git a/src/mailman/commands/cli_withlist.py b/src/mailman/commands/cli_withlist.py
index e7329f4e7..492391c7b 100644
--- a/src/mailman/commands/cli_withlist.py
+++ b/src/mailman/commands/cli_withlist.py
@@ -159,7 +159,7 @@ class Withlist:
     def _start_python(self, overrides, banner):
         # Set the tab completion.
         try:
-            import readline, rlcompleter            # flake8: noqa
+            import readline, rlcompleter            # noqa
             readline.parse_and_bind('tab: complete')
         except ImportError:
             pass
diff --git a/src/mailman/commands/eml_help.py b/src/mailman/commands/eml_help.py
index 8df4c666a..8166effea 100644
--- a/src/mailman/commands/eml_help.py
+++ b/src/mailman/commands/eml_help.py
@@ -66,7 +66,7 @@ class Help:
                 print(wrap(command.description), file=results)
             return ContinueProcessing.yes
         else:
-            printable_arguments = SPACE.join(arguments)   # flake8: noqa
+            printable_arguments = SPACE.join(arguments)   # noqa
             print(_('$self.name: too many arguments: $printable_arguments'),
                   file=results)
             return ContinueProcessing.no
diff --git a/src/mailman/commands/eml_membership.py b/src/mailman/commands/eml_membership.py
index 4f8e62a69..8168e98fa 100644
--- a/src/mailman/commands/eml_membership.py
+++ b/src/mailman/commands/eml_membership.py
@@ -92,7 +92,7 @@ used.
             return ContinueProcessing.yes
         joins.add(email)
         results.joins = joins
-        person = formataddr((display_name, email))   # flake8: noqa
+        person = formataddr((display_name, email))   # noqa
         # Is this person already a member of the list?  Search for all
         # matching memberships.
         members = getUtility(ISubscriptionService).find_members(
@@ -187,7 +187,7 @@ You may be asked to confirm your request.""")
                 file=results)
             return ContinueProcessing.no
         member.unsubscribe()
-        person = formataddr((user.display_name, email))   # flake8: noqa
+        person = formataddr((user.display_name, email))   # noqa
         print(_('$person left $mlist.fqdn_listname'), file=results)
         return ContinueProcessing.yes