Apply a few suggestions from refurb

src/virtualenv/app_data/na.py

@@ -53,7 +53,6 @@ def exists(self):
 
     def read(self):
         """Nothing to read."""
-        return
 
     def write(self, content):
         """Nothing to write."""

src/virtualenv/discovery/builtin.py

@@ -14,7 +14,7 @@
 class Builtin(Discover):
     def __init__(self, options) -> None:
         super().__init__(options)
-        self.python_spec = options.python if options.python else [sys.executable]
+        self.python_spec = options.python or [sys.executable]
         self.app_data = options.app_data
         self.try_first_with = options.try_first_with
 

src/virtualenv/discovery/py_spec.py

@@ -31,7 +31,7 @@ def from_string_spec(cls, string_spec):  # noqa: C901, PLR0912
             path = string_spec
         else:
             ok = False
-            match = re.match(PATTERN, string_spec)
+            match = PATTERN.match(string_spec)
             if match:
 
                 def _int_or_none(val):

tasks/make_zipapp.py

@@ -276,7 +276,7 @@ def get_wheels_for_support_versions(folder):
 class WheelForVersion:
     def __init__(self, wheel=None, versions=None) -> None:
         self.wheel = wheel
-        self.versions = versions if versions else []
+        self.versions = versions or []
 
     def __repr__(self) -> str:
         return f"{self.__class__.__name__}({self.wheel!r}, {self.versions!r})"

tests/conftest.py

@@ -187,7 +187,7 @@ def _check_os_environ_stable():
 
 
 COV_ENV_VAR = "COVERAGE_PROCESS_START"
-COVERAGE_RUN = os.environ.get(str(COV_ENV_VAR))
+COVERAGE_RUN = os.environ.get(COV_ENV_VAR)
 
 
 @pytest.fixture(autouse=True)