Skip to content

Commit

Permalink
Address flake8 E721 violations
Browse files Browse the repository at this point in the history
These violations don't show up in our CI right now because we've pinned
flake8<6, but when that pin is dropped, we'll have enforcement for this
rule.
  • Loading branch information
cottsay committed Sep 5, 2023
1 parent c8995e3 commit 34c2a1b
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion colcon_core/package_augmentation/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -213,7 +213,7 @@ def update_metadata(desc, key, value):
old_value |= value
return

if type(old_value) != type(value):
if type(old_value) is not type(value):
logger.warning(
f"update package '{desc.name}' metadata '{key}' from value "
f"'{old_value}' to '{value}'")
Expand Down
2 changes: 1 addition & 1 deletion colcon_core/package_descriptor.py
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ def __hash__(self): # noqa: D105
return hash((self.type, self.name))

def __eq__(self, other): # noqa: D105
if type(self) != type(other):
if type(self) is not type(other):
return NotImplemented
if (self.type, self.name) != (other.type, other.name):
return False
Expand Down
2 changes: 1 addition & 1 deletion colcon_core/verb/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,7 @@ def update_object(
return

severity = 5 \
if old_value is None or type(old_value) == type(value) \
if old_value is None or type(old_value) is type(value) \
else logging.WARNING
logger.log(
severity, f"overwrite package '{package_name}' {argument_type} "
Expand Down
4 changes: 2 additions & 2 deletions test/test_argument_default.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,13 @@ def test_argument_default():
unwrap_default_value(value)
default_value = wrap_default_value(value)
assert is_default_value(default_value)
assert type(default_value) != type(value)
assert type(default_value) is not type(value)
with pytest.raises(ValueError):
wrap_default_value(default_value)
unwrapped_value = unwrap_default_value(default_value)
assert value == unwrapped_value

value = 42
unchanged_value = wrap_default_value(value)
assert type(unchanged_value) == type(value)
assert type(unchanged_value) is type(value)
assert unchanged_value == value

0 comments on commit 34c2a1b

Please sign in to comment.