edwardwang888 commented on a change in pull request #18343:
URL: https://github.com/apache/airflow/pull/18343#discussion_r711687778
##########
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:
Yeah, it is not strictly necessary. I was just trying to produce the
closest boundary case 🙂
In general, is there a preference of whether to add (not strictly necessary)
libraries to the `devel` extra?
https://github.com/apache/airflow/blob/3f65a48bcedfab6a50e24ecac710170aab6f51bc/tests/models/test_taskinstance.py#L526-L532
--
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]
