Skip to content

Conditionally skip few more tests for "Pydantic Removed" test#38620

Merged
potiuk merged 1 commit intoapache:mainfrom
potiuk:skip-more-google-ai-tests
Mar 29, 2024
Merged

Conditionally skip few more tests for "Pydantic Removed" test#38620
potiuk merged 1 commit intoapache:mainfrom
potiuk:skip-more-google-ai-tests

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Mar 29, 2024

Turns out that we need to conditionally skip a few more tests in order to make "Pydantic Removed" tests happy after #38617


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

Turns out that we need to conditionally skip a few more tests in order to make
"Pydantic Removed" tests happy after apache#38617
@potiuk potiuk added the full tests needed We need to run full set of tests for this PR to merge label Mar 29, 2024
@boring-cyborg boring-cyborg bot added area:providers provider:google Google (including GCP) related issues labels Mar 29, 2024
@potiuk
Copy link
Member Author

potiuk commented Mar 29, 2024

Should fix main failing with:

Screenshot 2024-03-29 at 15 47 17

@potiuk potiuk merged commit 6e27ba9 into apache:main Mar 29, 2024
@potiuk potiuk deleted the skip-more-google-ai-tests branch March 29, 2024 16:42
@ephraimbuddy ephraimbuddy added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Mar 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) full tests needed We need to run full set of tests for this PR to merge provider:google Google (including GCP) related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants