Skip to content

Commit

Permalink
Fix sanity errors (#2297)
Browse files Browse the repository at this point in the history
SUMMARY

Fix sanity errors occurring with the devel branch of ansible
ERROR: Found 2 pylint issue(s) which need to be resolved:
ERROR: plugins/modules/ec2_vpc_endpoint.py:441:24: used-before-assignment: Using variable 'result' before assignment
ERROR: plugins/modules/ec2_vpc_route_table.py:792:7: used-before-assignment: Using variable 'route_table' before assignment

See #2295

ISSUE TYPE


Bugfix Pull Request
Docs Pull Request
Feature Pull Request
New Module Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell
  • Loading branch information
alinabuzachis committed Sep 18, 2024
1 parent b7770a5 commit 13d7be3
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 0 deletions.
2 changes: 2 additions & 0 deletions changelogs/fragments/20240919-sanity_fixes_devel.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
trivial:
- "Fix sanity errors occurring with the devel branch of ansible."
1 change: 1 addition & 0 deletions plugins/modules/ec2_vpc_endpoint.py
Original file line number Diff line number Diff line change
Expand Up @@ -419,6 +419,7 @@ def create_vpc_endpoint(client, module):

def setup_removal(client, module):
params = dict()
result = {}
changed = False

if module.check_mode:
Expand Down
1 change: 1 addition & 0 deletions plugins/modules/ec2_vpc_route_table.py
Original file line number Diff line number Diff line change
Expand Up @@ -763,6 +763,7 @@ def ensure_route_table_present(connection, module):

changed = False
tags_valid = False
route_table = None

if lookup == "tag":
if tags is not None:
Expand Down

0 comments on commit 13d7be3

Please sign in to comment.