Merge "Fix F841 and remove exclusion"

plugins/modules/os_project_access.py

@@ -121,7 +121,6 @@ def main():
 
     sdk, cloud = openstack_cloud_from_module(module)
 
-    changed = False
     state = module.params['state']
     resource_name = module.params['resource_name']
     resource_type = module.params['resource_type']

plugins/modules/os_router.py

@@ -329,7 +329,6 @@ def _validate_subnets(module, cloud, filters=None):
     internal_subnet_ids = []
     internal_port_ids = []
     existing_port_ips = []
-    existing_port_ids = []
     if module.params['external_fixed_ips']:
         for iface in module.params['external_fixed_ips']:
             subnet = cloud.get_subnet(iface['subnet'])

tox.ini

@@ -51,7 +51,7 @@ commands = {posargs}
 # H4 are rules for docstrings. Maybe we should clean them?
 # E501,E402,H301,H236,F401,E128 are ignored so we can import the existing
 #      modules unchanged and then clean them in subsequent patches.
-ignore = W503,H4,E501,E402,H301,H236,F401,E128,W504,F841,F403,F405
+ignore = W503,H4,E501,E402,H301,H236,F401,E128,W504,F403,F405
 show-source = True
 exclude=.venv,.git,.tox,dist,doc,*lib/python*,*egg,build,ansible_collections