Export the non harmful g-import-not-at-top pylint pragmas.

yilei · copybara-github · commit ba6b5ad912f7 · 2023-04-18T15:46:27.000-07:00
PiperOrigin-RevId: 525272674
Change-Id: I4308a1f615693655818f95e2a413f38eebb3d872
diff --git a/absl/app.py b/absl/app.py
@@ -238,6 +238,7 @@ def _run_main(main, argv):
   elif FLAGS.run_with_profiling or FLAGS.profile_file:
     # Avoid import overhead since most apps (including performance-sensitive
     # ones) won't be run with profiling.
+    # pylint: disable=g-import-not-at-top
     import atexit
     if FLAGS.use_cprofile_for_profiling:
       import cProfile as profile
diff --git a/absl/command_name.py b/absl/command_name.py
@@ -47,7 +47,7 @@ def set_kernel_process_name(name):
       proc_comm.write(name[:15])
   except EnvironmentError:
     try:
-      import ctypes
+      import ctypes  # pylint: disable=g-import-not-at-top
     except ImportError:
       return  # No ctypes.
     try:
diff --git a/absl/flags/_argument_parser.py b/absl/flags/_argument_parser.py
@@ -386,7 +386,7 @@ def __init__(self, enum_class, case_sensitive=True):
     """
     # Users must have an Enum class defined before using EnumClass flag.
     # Therefore this dependency is guaranteed.
-    import enum
+    import enum  # pylint: disable=g-import-not-at-top
 
     if not issubclass(enum_class, enum.Enum):
       raise TypeError('{} is not a subclass of Enum.'.format(enum_class))
diff --git a/absl/flags/_defines.py b/absl/flags/_defines.py
@@ -27,6 +27,7 @@
 from absl.flags import _helpers
 from absl.flags import _validators
 
+# pylint: disable=g-import-not-at-top
 # pylint: disable=unused-import
 try:
   from typing import Text, List, Any
@@ -37,6 +38,7 @@
   import enum
 except ImportError:
   pass
+# pylint: enable=g-import-not-at-top
 # pylint: enable=unused-import
 
 _helpers.disclaim_module_ids.add(id(sys.modules[__name__]))
diff --git a/absl/flags/_helpers.py b/absl/flags/_helpers.py
@@ -20,6 +20,7 @@
 import struct
 import sys
 import textwrap
+# pylint: disable=g-import-not-at-top
 try:
   import fcntl
 except ImportError:
@@ -29,6 +30,7 @@
   import termios
 except ImportError:
   termios = None
+# pylint: enable=g-import-not-at-top
 
 
 _DEFAULT_HELP_WIDTH = 80  # Default width of help output.
diff --git a/absl/flags/tests/flags_test.py b/absl/flags/tests/flags_test.py
@@ -993,7 +993,7 @@ def args_list():
     # to be raised.
     try:
       sys.modules.pop('absl.flags.tests.module_baz')
-      import absl.flags.tests.module_baz
+      import absl.flags.tests.module_baz  # pylint: disable=g-import-not-at-top
       del absl
     except flags.DuplicateFlagError:
       raise AssertionError('Module reimport caused flag duplication error')
diff --git a/absl/logging/__init__.py b/absl/logging/__init__.py
@@ -97,11 +97,13 @@
 from absl import flags
 from absl.logging import converter
 
+# pylint: disable=g-import-not-at-top
 try:
   from typing import NoReturn
 except ImportError:
   pass
 
+# pylint: enable=g-import-not-at-top
 
 FLAGS = flags.FLAGS
 
diff --git a/absl/logging/tests/verbosity_flag_test.py b/absl/logging/tests/verbosity_flag_test.py
@@ -27,9 +27,11 @@
     'logging.root level should be changed to ERROR, but found {}'.format(
         logging.root.getEffectiveLevel()))
 
+# pylint: disable=g-import-not-at-top
 from absl import flags
 from absl import logging as _  # pylint: disable=unused-import
 from absl.testing import absltest
+# pylint: enable=g-import-not-at-top
 
 FLAGS = flags.FLAGS
 
diff --git a/absl/testing/absltest.py b/absl/testing/absltest.py
@@ -47,18 +47,19 @@
   # The faulthandler module isn't always available, and pytype doesn't
   # understand that we're catching ImportError, so suppress the error.
   # pytype: disable=import-error
-  import faulthandler
+  import faulthandler  # pylint: disable=g-import-not-at-top
   # pytype: enable=import-error
 except ImportError:
   # We use faulthandler if it is available.
   faulthandler = None
 
-from absl import app
+from absl import app  # pylint: disable=g-import-not-at-top
 from absl import flags
 from absl import logging
 from absl.testing import _pretty_print_reporter
 from absl.testing import xml_reporter
 
+# pylint: disable=g-import-not-at-top
 # Make typing an optional import to avoid it being a required dependency
 # in Python 2. Type checkers will still understand the imports.
 try:
@@ -79,6 +80,7 @@
     _OutcomeType = unittest.case._Outcome  # pytype: disable=module-attr
 
 
+# pylint: enable=g-import-not-at-top
 
 # Re-export a bunch of unittest functions we support so that people don't
 # have to import unittest to get them
diff --git a/absl/tests/app_test_helper.py b/absl/tests/app_test_helper.py
@@ -18,11 +18,11 @@
 import sys
 
 try:
-  import faulthandler
+  import faulthandler  # pylint: disable=g-import-not-at-top
 except ImportError:
   faulthandler = None
 
-from absl import app
+from absl import app  # pylint: disable=g-import-not-at-top
 from absl import flags
 
 FLAGS = flags.FLAGS
diff --git a/setup.py b/setup.py
@@ -17,12 +17,14 @@
 import os
 import sys
 
+# pylint: disable=g-import-not-at-top
 try:
   import setuptools
 except ImportError:
   from ez_setup import use_setuptools
   use_setuptools()
   import setuptools
+# pylint: enable=g-import-not-at-top
 
 if sys.version_info < (3, 7):
   raise RuntimeError('Python version 3.7+ is required.')
