Skip to content

Sync with 3.29.0 #406

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Feb 23, 2025
Merged

Sync with 3.29.0 #406

merged 7 commits into from
Feb 23, 2025

Conversation

dkropachev
Copy link
Collaborator

@dkropachev dkropachev commented Jan 5, 2025

Sync with 3.29.0 of the upstream

One significant refactor needed to make it work, after c44e264 multiple test become broken, this commit introduced DependencyException for missing requirements for asyncorereactor.py and libevreactor.py and partially patched code that
imports them, as result many tests where broken.
I have created following up commit that addresses all the problems: 47d882d

Pre-review checklist

  • I have split my patch into logically separate commits.
  • All commit messages clearly explain what they change and why.
  • I added relevant tests for new features and bug fixes.
  • All commits compile, pass static checks and pass test.
  • PR description sums up the changes and reasons why they should be introduced.
  • I have provided docstrings for the public items that I want to introduce.
  • I have adjusted the documentation in ./docs/source/.
  • I added appropriate Fixes: annotations to PR description.

@dkropachev dkropachev changed the title Dk/sync with 3.29.0 Sync with 3.29.0 Jan 5, 2025
@dkropachev dkropachev force-pushed the dk/sync-with-3.29.0 branch 3 times, most recently from 270d2f5 to d9ffed0 Compare January 5, 2025 16:15
@dkropachev dkropachev force-pushed the dk/sync-with-3.29.0 branch 2 times, most recently from e0f3caf to 85999e3 Compare February 21, 2025 20:36
@dkropachev dkropachev force-pushed the dk/sync-with-3.29.0 branch 11 times, most recently from 1eedd4a to 5d0a93d Compare February 22, 2025 15:49
@dkropachev dkropachev marked this pull request as ready for review February 22, 2025 17:53
@dkropachev dkropachev changed the title Sync with 3.29.0 WIP: Sync with 3.29.0 Feb 23, 2025
Copy link

@fruch fruch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dkropachev dkropachev force-pushed the dk/sync-with-3.29.0 branch 4 times, most recently from 7ef3a9c to 3c06734 Compare February 23, 2025 13:00
@dkropachev dkropachev force-pushed the dk/sync-with-3.29.0 branch 2 times, most recently from 551f37f to 85a40ba Compare February 23, 2025 13:29
@dkropachev dkropachev changed the title WIP: Sync with 3.29.0 Sync with 3.29.0 Feb 23, 2025
@dkropachev dkropachev merged commit a226518 into master Feb 23, 2025
21 checks passed
@roydahan
Copy link
Collaborator

Cheers!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants