You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/01/10 07:28:36 UTC

[GitHub] [airflow] dstandish opened a new pull request #7124: [AIRFLOW-6528] disable flake8 W503 line break before binary operator

dstandish opened a new pull request #7124: [AIRFLOW-6528] disable flake8 W503 line break before binary operator
URL: https://github.com/apache/airflow/pull/7124
 
 
   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:
   ```
       @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
           )
   ```
   
   as required by black (W503 disabled)
   ```
       @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
           )
   ```
   
   ---
   Issue link: WILL BE INSERTED BY [boring-cyborg](https://github.com/kaxil/boring-cyborg)
   
   - [x] Description above provides context of the change
   - [x] Commit message/PR title starts with `[AIRFLOW-NNNN]`. AIRFLOW-NNNN = JIRA ID<sup>*</sup>
   - [x] Unit tests coverage for changes (not needed for documentation changes)
   - [x] Commits follow "[How to write a good git commit message](http://chris.beams.io/posts/git-commit/)"
   - [x] Relevant documentation is updated including usage instructions.
   - [x] I will engage committers as explained in [Contribution Workflow Example](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#contribution-workflow-example).
   
   <sup>*</sup> For document-only changes commit message can start with `[AIRFLOW-XXXX]`.
   
   ---
   In case of fundamental code change, Airflow Improvement Proposal ([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvements+Proposals)) is needed.
   In case of a new dependency, check compliance with the [ASF 3rd Party License Policy](https://www.apache.org/legal/resolved.html#category-x).
   In case of backwards incompatible changes please leave a note in [UPDATING.md](https://github.com/apache/airflow/blob/master/UPDATING.md).
   Read the [Pull Request Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines) for more information.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] potiuk commented on issue #7124: [AIRFLOW-6528] disable flake8 W503 line break before binary operator

Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #7124: [AIRFLOW-6528] disable flake8 W503 line break before binary operator
URL: https://github.com/apache/airflow/pull/7124#issuecomment-573097121
 
 
   Thanks @dstandish !

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] potiuk merged pull request #7124: [AIRFLOW-6528] disable flake8 W503 line break before binary operator

Posted by GitBox <gi...@apache.org>.
potiuk merged pull request #7124: [AIRFLOW-6528] disable flake8 W503 line break before binary operator
URL: https://github.com/apache/airflow/pull/7124
 
 
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [airflow] potiuk commented on issue #7124: [AIRFLOW-6528] disable flake8 W503 line break before binary operator

Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #7124: [AIRFLOW-6528] disable flake8 W503 line break before binary operator
URL: https://github.com/apache/airflow/pull/7124#issuecomment-572923433
 
 
   @mik-laj - what do you think - I am ok with it :)

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services