Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Jul 29, 2023

After youtype/mypy_boto3_builder#209
has been fixed and mypy-boto3-appflow released, we can now
remove the limit and intrduce lower-bound limit for it.


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

@potiuk
Copy link
Member Author

potiuk commented Jul 29, 2023

Just testing if post1 fixes the problem

@potiuk potiuk force-pushed the test-post1-fix-of-appflow branch from 695686d to 39734f1 Compare August 4, 2023 16:33
@potiuk potiuk changed the title Test post1 version of appflow Update mypy-boto3-appflow dependency Aug 4, 2023
@potiuk potiuk marked this pull request as ready for review August 4, 2023 16:33
@potiuk potiuk requested review from ferruzzi and vincbeck August 4, 2023 16:33
@potiuk potiuk force-pushed the test-post1-fix-of-appflow branch from 39734f1 to 9374d03 Compare August 4, 2023 21:00
@o-nikolas
Copy link
Contributor

Changes look good to me, but there are tests failing. I'm going to re-trigger those now, I suspect the failures are spurious.

@uranusjr
Copy link
Member

uranusjr commented Aug 5, 2023

The 3.11 failures seem to be consistent.

@potiuk
Copy link
Member Author

potiuk commented Aug 5, 2023

The 3.11 failures seem to be consistent.

Yeah. It also happened in main. This was caused by some dependency in 3.11 not pulling future library and one of our examples using import from future for urllib/urlsplit. Fixed in #33137

Rebasing to apply it to this PR.

After youtype/mypy_boto3_builder#209
has been fixed and mypy-boto3-appflow released, we can now
remove the limit and intrduce lower-bound limit for it.
@potiuk potiuk force-pushed the test-post1-fix-of-appflow branch from 9374d03 to 294a0c4 Compare August 5, 2023 10:46
@potiuk potiuk merged commit 46862cf into apache:main Aug 6, 2023
@potiuk potiuk deleted the test-post1-fix-of-appflow branch August 6, 2023 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants