Skip to content

Conversation

@sandrobonazzola
Copy link
Contributor

@sandrobonazzola sandrobonazzola commented Jun 17, 2025

Add testing with Python 3.14: includes fixes for #527 needed to pass.

@mergify
Copy link
Contributor

mergify bot commented Jun 17, 2025

⚠️ No release notes detected. Please make sure to use reno to add a changelog entry.

```
tenacity/__init__.py:314: error: Returning Any from function declared to return "dict[str, Any]"  [no-any-return]
tenacity/__init__.py:322: error: Returning Any from function declared to return "IterState"  [no-any-return]
Found 2 errors in 1 file (checked 19 source files)
```

Signed-off-by: Sandro Bonazzola <sandro.bonazzola@gmail.com>
This refactors the original code to work with Python 3.14,
leveraging `asyncio.run()` for automatic event loop management,
which is the recommended approach in modern asyncio.

Fixes jd#527.

Signed-off-by: Sandro Bonazzola <sandro.bonazzola@gmail.com>
Related-To: jd#527
Signed-off-by: Sandro Bonazzola <sandro.bonazzola@gmail.com>
@sandrobonazzola sandrobonazzola changed the title Test with Python 3.14 Support Python 3.14 Jun 17, 2025
Signed-off-by: Sandro Bonazzola <sandro.bonazzola@gmail.com>
@sandrobonazzola
Copy link
Contributor Author

Addressed mergify comment. @jd anything else missing?

@sandrobonazzola
Copy link
Contributor Author

@jd is mergify supposed to handle the merge after your approval? It seems it's not really doing it.

@jd
Copy link
Owner

jd commented Jun 26, 2025

@jd is mergify supposed to handle the merge after your approval? It seems it's not really doing it.

#528 (comment)

@sandrobonazzola
Copy link
Contributor Author

@jd is mergify supposed to handle the merge after your approval? It seems it's not really doing it.

#528 (comment)

But release notes are included: 561c370

@sandrobonazzola
Copy link
Contributor Author

@Mergifyio refresh

@mergify
Copy link
Contributor

mergify bot commented Jun 27, 2025

refresh

✅ Pull request refreshed

@sandrobonazzola
Copy link
Contributor Author

@jd looking at .mergify.yml I think you need to manually add this to the merge queue. Seems like only your PRs and dependabot PRs are added to the merge queue. But maybe I'm wrong, it's the first time I see mergify.

@jd
Copy link
Owner

jd commented Jun 27, 2025

It's just that you changed the CI list, so it can't be merged automatically, I missed that.

@jd jd merged commit eed7d78 into jd:main Jun 27, 2025
10 checks passed
@sandrobonazzola
Copy link
Contributor Author

Thanks!

@sandrobonazzola sandrobonazzola deleted the 3.14_test branch June 27, 2025 08:20
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.

2 participants