dirrao commented on code in PR #39908:
URL: https://github.com/apache/airflow/pull/39908#discussion_r1691675192
##########
tests/_internals/forbidden_warnings.py:
##########
@@ -62,6 +62,11 @@ def pytest_itemcollected(self, item: pytest.Item):
# Add marker at the beginning of the markers list. In this case,
it does not conflict with
# filterwarnings markers, which are set explicitly in the test
suite.
item.add_marker(pytest.mark.filterwarnings(f"error::{fw}"),
append=False)
+ item.add_marker(
Review Comment:
> Hi @potiuk / @kaxil,
> I am looking for the comment on the above suggestion "Using seconds
instead of milli seconds" from @ferruzzi.
> Please let us know your thoughts on this.
@kaxil Can you share your comments on this? So, that we all align on it and
will make the necessary changes.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
