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 2022/07/02 06:41:24 UTC

[GitHub] [airflow] BasPH opened a new pull request, #24797: Replace all NBSP characters by whitespaces

BasPH opened a new pull request, #24797:
URL: https://github.com/apache/airflow/pull/24797

   This PR replaces all NBSP characters by whitespaces. Not visible in GitHub, but e.g. PyCharm displays them:
   
   ![Greenshot 2022-07-02 08 32 20](https://user-images.githubusercontent.com/6249654/176989723-87f31419-cd78-4c53-ba6b-2a7ae308b9c5.png)
   
   NBSP characters can lead to hard to discover issues when copy-pasting.
   
   ---
   **^ Add meaningful description above**
   
   Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)** for more information.
   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 a newsfragement file, named `{pr_number}.significant.rst`, in [newsfragments](https://github.com/apache/airflow/tree/main/newsfragments).
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] potiuk commented on pull request #24797: Replace all NBSP characters by whitespaces

Posted by GitBox <gi...@apache.org>.
potiuk commented on PR #24797:
URL: https://github.com/apache/airflow/pull/24797#issuecomment-1174828683

   > Can we add pre commit to avoid future cases?
   
   Good point


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] potiuk merged pull request #24797: Replace all NBSP characters by whitespaces

Posted by GitBox <gi...@apache.org>.
potiuk merged PR #24797:
URL: https://github.com/apache/airflow/pull/24797


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] potiuk commented on pull request #24797: Replace all NBSP characters by whitespaces

Posted by GitBox <gi...@apache.org>.
potiuk commented on PR #24797:
URL: https://github.com/apache/airflow/pull/24797#issuecomment-1173438918

   Some copy&paste from word :)  likely


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org