potiuk commented on a change in pull request #8487:
URL: https://github.com/apache/airflow/pull/8487#discussion_r412154451
##########
File path: setup.py
##########
@@ -446,6 +446,9 @@ def write_version(filename: str = os.path.join(*[my_dir,
"airflow", "git_version
'pytest',
'pytest-cov',
'pytest-instafail',
+ 'pytest-rerunfailures',
Review comment:
Yes. I plan to add it in a follow-up PR. I did not not want to
drag-along all the changes as I was asked to split the PR into smaller and I
would be able to only add --rerun option at the very end where I added
quarantine option in #8393 . I figured also based on the comment from
@turbaszek https://github.com/apache/airflow/pull/8393#discussion_r411108454
that in this case it's better to split to a "adding new requirements" and
"using the new requirements".
Do you think @kaxil it shoudl be merged instead? I am ok to do either. But I
think it's a bit cleaner this way.
----------------------------------------------------------------
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:
[email protected]
