[
https://issues.apache.org/jira/browse/AIRFLOW-6528?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17013018#comment-17013018
]
ASF subversion and git services commented on AIRFLOW-6528:
----------------------------------------------------------
Commit d6170e81368e4edc2f0594a559e97b60898120a4 in airflow's branch
refs/heads/master from Daniel Standish
[ https://gitbox.apache.org/repos/asf?p=airflow.git;h=d6170e8 ]
[AIRFLOW-6528] disable flake8 W503 line break before binary operator (#7124)
> disable W503 flake8 check (line break before binary operator)
> -------------------------------------------------------------
>
> Key: AIRFLOW-6528
> URL: https://issues.apache.org/jira/browse/AIRFLOW-6528
> Project: Apache Airflow
> Issue Type: Bug
> Components: pre-commit
> Affects Versions: 1.10.7
> Reporter: Daniel Standish
> Priority: Trivial
>
> Flake8's W503 rule says there should be no line break before binary operator.
> This rule is incompatible with black formatter, and is also in my opinion bad
> style.
> Status quo example with W503 check enabled:
> {code}
> @property
> def sqlalchemy_scheme(self):
> """
> Database provided in init if exists; otherwise, ``schema`` from
> ``Connection`` object.
> """
> return (
> self._sqlalchemy_scheme or
> self.connection_extra_lower.get('sqlalchemy_scheme') or
> self.DEFAULT_SQLALCHEMY_SCHEME
> )
> {code}
> as required by black (W503 disabled)
> {code}
> @property
> def sqlalchemy_scheme(self):
> """
> Database provided in init if exists; otherwise, ``schema`` from
> ``Connection`` object.
> """
> return (
> self._sqlalchemy_scheme
> or self.connection_extra_lower.get('sqlalchemy_scheme')
> or self.DEFAULT_SQLALCHEMY_SCHEME
> )
> {code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
