Follow-up from "Fix lint issues": Avoid the need to disable `assignment-from-no-return` for abstract methods
The following discussion from !892 (merged) should be addressed:
-
@juergbi started a discussion: (+3 comments) We can probably get rid of the handful of
# pylint: disable=assignment-from-no-returnby using @abc around the code base. Looking at how many of those exceptions we need, this might well be worthwhile. Definitely a new MR/issue, but still worth noting.I don't like requiring
disable=assignment-from-no-returnfor this common pattern. If@abcindeed works for this, shouldn't this be a rather simple improvement and better done as part of this MR?