Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Dec 4, 2023

Seems that the pytest-asyncio problem was already introduced by 0.23.0. In order to allow tests passing now, we should limit it to below that version (follow up after #36037)


^ 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.

Seems that the pytest-asyncio problem was already introduced by
0.23.0. In order to allow tests passing now, we should limit it
to below that version (follow up after apache#36037)
@potiuk potiuk requested a review from eladkal December 4, 2023 01:35
@potiuk
Copy link
Member Author

potiuk commented Dec 4, 2023

Yes. Confirmed that <0.23.0 solves the problem.

@eladkal eladkal merged commit cc2521c into apache:main Dec 4, 2023
@ephraimbuddy ephraimbuddy added this to the Airflow 2.8.0 milestone Dec 5, 2023
@ephraimbuddy ephraimbuddy added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Dec 5, 2023
ephraimbuddy pushed a commit that referenced this pull request Dec 5, 2023
Seems that the pytest-asyncio problem was already introduced by
0.23.0. In order to allow tests passing now, we should limit it
to below that version (follow up after #36037)

(cherry picked from commit cc2521c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants