diff --git a/pandas/io/parsers/python_parser.py b/pandas/io/parsers/python_parser.py index daf89319a4520..ddd73375f8871 100644 --- a/pandas/io/parsers/python_parser.py +++ b/pandas/io/parsers/python_parser.py @@ -704,7 +704,7 @@ def _next_line(self) -> list[Scalar]: self._is_line_empty(self.data[self.pos - 1]) or line ): break - elif self.skip_blank_lines: + if self.skip_blank_lines: ret = self._remove_empty_lines([line]) if ret: line = ret[0] diff --git a/pandas/io/sql.py b/pandas/io/sql.py index 783b5f8cd63ae..0f24e3f31cc4b 100644 --- a/pandas/io/sql.py +++ b/pandas/io/sql.py @@ -971,18 +971,18 @@ def _query_iterator( [], columns=columns, coerce_float=coerce_float ) break - else: - has_read_data = True - self.frame = DataFrame.from_records( - data, columns=columns, coerce_float=coerce_float - ) - self._harmonize_columns(parse_dates=parse_dates) + has_read_data = True + self.frame = DataFrame.from_records( + data, columns=columns, coerce_float=coerce_float + ) - if self.index is not None: - self.frame.set_index(self.index, inplace=True) + self._harmonize_columns(parse_dates=parse_dates) + + if self.index is not None: + self.frame.set_index(self.index, inplace=True) - yield self.frame + yield self.frame def read( self, @@ -1489,16 +1489,16 @@ def _query_iterator( parse_dates=parse_dates, ) break - else: - has_read_data = True - yield _wrap_result( - data, - columns, - index_col=index_col, - coerce_float=coerce_float, - parse_dates=parse_dates, - dtype=dtype, - ) + + has_read_data = True + yield _wrap_result( + data, + columns, + index_col=index_col, + coerce_float=coerce_float, + parse_dates=parse_dates, + dtype=dtype, + ) def read_query( self, @@ -2053,16 +2053,16 @@ def _query_iterator( [], columns=columns, coerce_float=coerce_float ) break - else: - has_read_data = True - yield _wrap_result( - data, - columns, - index_col=index_col, - coerce_float=coerce_float, - parse_dates=parse_dates, - dtype=dtype, - ) + + has_read_data = True + yield _wrap_result( + data, + columns, + index_col=index_col, + coerce_float=coerce_float, + parse_dates=parse_dates, + dtype=dtype, + ) def read_query( self, diff --git a/pandas/plotting/_matplotlib/converter.py b/pandas/plotting/_matplotlib/converter.py index 70fca27c837f3..4d5feafb5ebd2 100644 --- a/pandas/plotting/_matplotlib/converter.py +++ b/pandas/plotting/_matplotlib/converter.py @@ -404,9 +404,8 @@ def __call__(self): if num <= interval * (max_millis_ticks - 1): self._interval = interval break - else: - # We went through the whole loop without breaking, default to 1 - self._interval = 1000.0 + # We went through the whole loop without breaking, default to 1 + self._interval = 1000.0 estimate = (nmax - nmin) / (self._get_unit() * self._get_interval()) diff --git a/pandas/tests/io/test_user_agent.py b/pandas/tests/io/test_user_agent.py index ac4ca5dce6dc3..3b552805198b5 100644 --- a/pandas/tests/io/test_user_agent.py +++ b/pandas/tests/io/test_user_agent.py @@ -233,9 +233,8 @@ def responder(request): if wait_time > kill_time: server_process.kill() break - else: - wait_time += 0.1 - time.sleep(0.1) + wait_time += 0.1 + time.sleep(0.1) server_process.close() diff --git a/pyproject.toml b/pyproject.toml index 09ea61c06380c..f18b54a8dcd03 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -122,7 +122,6 @@ disable = [ "inconsistent-return-statements", "invalid-sequence-index", "literal-comparison", - "no-else-break", "no-else-continue", "no-else-raise", "no-else-return",