VladaZakharova opened a new pull request, #36473:
URL: https://github.com/apache/airflow/pull/36473
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guid
Lee-W commented on code in PR #36473:
URL: https://github.com/apache/airflow/pull/36473#discussion_r1437972926
##
tests/system/providers/google/cloud/automl/example_automl_video_intelligence_classification.py:
##
@@ -73,73 +72,88 @@
schedule="@once",
start_date=datetim
VladaZakharova commented on code in PR #36473:
URL: https://github.com/apache/airflow/pull/36473#discussion_r1445769655
##
tests/system/providers/google/cloud/automl/example_automl_video_intelligence_tracking.py:
##
@@ -22,49 +22,49 @@
import os
from datetime import datetime
Lee-W commented on code in PR #36473:
URL: https://github.com/apache/airflow/pull/36473#discussion_r1448363959
##
tests/system/providers/google/cloud/automl/example_automl_nl_text_classification.py:
##
@@ -114,7 +112,7 @@
import_configs=DATA_CONFIG,
)
-# [STA
VladaZakharova commented on code in PR #36473:
URL: https://github.com/apache/airflow/pull/36473#discussion_r1448466822
##
tests/system/providers/google/cloud/automl/example_automl_nl_text_classification.py:
##
@@ -114,7 +112,7 @@
import_configs=DATA_CONFIG,
)
-
potiuk commented on code in PR #36473:
URL: https://github.com/apache/airflow/pull/36473#discussion_r1451813905
##
tests/system/providers/google/cloud/automl/example_automl_nl_text_classification.py:
##
@@ -114,7 +112,7 @@
import_configs=DATA_CONFIG,
)
-# [ST
VladaZakharova commented on code in PR #36473:
URL: https://github.com/apache/airflow/pull/36473#discussion_r1452059077
##
tests/system/providers/google/cloud/automl/example_automl_nl_text_classification.py:
##
@@ -114,7 +112,7 @@
import_configs=DATA_CONFIG,
)
-
VladaZakharova commented on PR #36473:
URL: https://github.com/apache/airflow/pull/36473#issuecomment-1893435045
Hi!
Can you please check this PR? :)
The service will be removed at the end of the month, so will be great to
deprecate it at least :) Thank you!
--
This is an automated
Lee-W commented on PR #36473:
URL: https://github.com/apache/airflow/pull/36473#issuecomment-1893909929
If the `text` part is deprecated as well, should we added it to PR title?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
VladaZakharova commented on PR #36473:
URL: https://github.com/apache/airflow/pull/36473#issuecomment-1893939681
> If the `text` part is deprecated as well, should we added it to PR title?
Text was deprecated in another PR, long time ago :)
--
This is an automated message from the A
Lee-W commented on PR #36473:
URL: https://github.com/apache/airflow/pull/36473#issuecomment-1894714786
> > If the `text` part is deprecated as well, should we added it to PR title?
>
> Text was deprecated in another PR, long time ago :)
Ah, yes, you're right. Missed the deleted
VladaZakharova commented on PR #36473:
URL: https://github.com/apache/airflow/pull/36473#issuecomment-1895523166
@eladkal
Hi! Can you please check the PR? Thanks! :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
potiuk merged PR #36473:
URL: https://github.com/apache/airflow/pull/36473
--
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-unsubscr...@airflow.a
13 matches
Mail list logo
