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/11 05:31:48 UTC
[GitHub] [airflow] edithturn opened a new pull request, #24955: D400 first line should end with period two files batch 01
edithturn opened a new pull request, #24955:
URL: https://github.com/apache/airflow/pull/24955
Related to the issue: #10742
This PR is the first of many that I will send to fix the D400 | First line should end with a period (That is ignoring in pre-commits).
I worked over two files: 0046_1_10_5_change_datetime_to_datetime2_6_on_mssql_.py and 0113_2_4_0_compare_types_between_orm_and_db.py , adding a period at the end of the first line and Giving the first line a shorter title if it doesn't already have one.
--
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] edithturn commented on pull request #24955: D400 first line should end with period two files batch 01
Posted by GitBox <gi...@apache.org>.
edithturn commented on PR #24955:
URL: https://github.com/apache/airflow/pull/24955#issuecomment-1180633964
I am not sure what I did wrong, But now I know how I will work on this issue, this is just an example about the first batch, I am splitting and it will be better to work on that issue by fixing the small number of files each time, maybe more fixes next time 💃🏼 💪🏼
--
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 #24955: D400 first line should end with period two files batch 01
Posted by GitBox <gi...@apache.org>.
potiuk commented on PR #24955:
URL: https://github.com/apache/airflow/pull/24955#issuecomment-1180323143
Smth. wrong- does not seem there are any changes here.
--
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 #24955: D400 first line should end with period two files batch 01
Posted by GitBox <gi...@apache.org>.
potiuk merged PR #24955:
URL: https://github.com/apache/airflow/pull/24955
--
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] edithturn commented on pull request #24955: D400 first line should end with period two files batch 01
Posted by GitBox <gi...@apache.org>.
edithturn commented on PR #24955:
URL: https://github.com/apache/airflow/pull/24955#issuecomment-1182734873
Thank you, Jarek! 💪🏼
--
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