Skip to content

Commit f452453

Browse files
committed
fix clippy and make tests meaningful
1 parent d5fb0cb commit f452453

File tree

2 files changed

+4
-1
lines changed
  • crates/ruff_linter

2 files changed

+4
-1
lines changed

crates/ruff_linter/resources/test/fixtures/flake8_return/RET501.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,15 @@ def prop(self) -> None:
3333

3434
@abc.abstractproperty
3535
def prop2(self) -> None:
36+
print("Override me")
3637
return None
3738

3839
@types.DynamicClassAttribute
3940
def prop3(self) -> None:
41+
print("Gotta make this a multiline function for it to be a meaningful test")
4042
return None
4143

4244
@enum.property
4345
def prop4(self) -> None:
46+
print("I've run out of things to say")
4447
return None

crates/ruff_linter/src/rules/flake8_return/rules/function.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -391,7 +391,7 @@ fn unnecessary_return_none(checker: &mut Checker, decorator_list: &[Decorator],
391391
}
392392

393393
/// Determine whether `decorator` is `@property`,
394-
/// or another stdlib decorator similar to `@property.
394+
/// or another stdlib decorator similar to `@property`.
395395
///
396396
/// TODO: be more principled here once we have type inference;
397397
/// hardcoding these is a little hacky.

0 commit comments

Comments
 (0)