mik-laj commented on a change in pull request #18343:
URL: https://github.com/apache/airflow/pull/18343#discussion_r711648377
##########
File path: tests/models/test_taskinstance.py
##########
@@ -17,15 +17,16 @@
# under the License.
import datetime
+import math
import os
import signal
-import time
import urllib
from tempfile import NamedTemporaryFile
from typing import List, Optional, Union, cast
from unittest import mock
from unittest.mock import call, mock_open, patch
+import numpy as np
Review comment:
This library is not an Airflow dependency. If we want to use it for
core testing, it should be installed with `deve` extral.
https://github.com/apache/airflow/blob/main/setup.py#L495
Or maybe you have an idea how to write this test without using this library?
--
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]
