Skip to content

fix attrs==25.2.0 compatibility #13291

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 1 commit into from
Mar 13, 2025
Merged

fix attrs==25.2.0 compatibility #13291

merged 1 commit into from
Mar 13, 2025

Conversation

jakkdl
Copy link
Member

@jakkdl jakkdl commented Mar 12, 2025

fixes #13290

@jakkdl
Copy link
Member Author

jakkdl commented Mar 12, 2025

does this need a newsfragment?

@jakkdl jakkdl requested a review from The-Compiler March 12, 2025 14:13
@webknjaz
Copy link
Member

Well, this seems user-facing. So I'd say yes.

@The-Compiler
Copy link
Member

Not user-facing in the sense of "people using pytest". However, we have a contrib type for news entries, and it would probably fit there (from changelog/README.rst):

  • contrib: stuff that affects the contributor experience. e.g. Running tests, building the docs, setting up the development environment.

@nicoddemus
Copy link
Member

I agree with @webknjaz that this is user facing, because without this fix, pytest support for attrs==25.2.0 does not work properly.

I suggest 13291.bugfix.rst:

Fixed support for ``attrs 25.2``.

@Pierre-Sassoulas
Copy link
Member

I also think it's user facing. Maybe 'fixed repr of attr object in assertion for attr 25.2' ?

@The-Compiler
Copy link
Member

Oh, right. Somehow I assumed this was only in the self-test file. My bad!

@bluetech
Copy link
Member

In order to unbreak CI I'll add a changelog based on @nicoddemus and @Pierre-Sassoulas suggestions and merge.

@psf-chronographer psf-chronographer bot added the bot:chronographer:provided (automation) changelog entry is part of PR label Mar 13, 2025
@bluetech bluetech merged commit 4933d92 into pytest-dev:main Mar 13, 2025
28 checks passed
Copy link

patchback bot commented Mar 13, 2025

Backport to 8.3.x: 💚 backport PR created

✅ Backport PR branch: patchback/backports/8.3.x/4933d9223ffb3e9c1f11a4a2623be5d32672e071/pr-13291

Backported as #13294

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Mar 13, 2025
fix attrs==25.2.0 compatibility

(cherry picked from commit 4933d92)
@jakkdl jakkdl deleted the fix_attrs branch March 13, 2025 12:32
bluetech added a commit that referenced this pull request Mar 13, 2025
…933d9223ffb3e9c1f11a4a2623be5d32672e071/pr-13291

[PR #13291/4933d922 backport][8.3.x] fix attrs==25.2.0 compatibility
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot:chronographer:provided (automation) changelog entry is part of PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI fails with attrs 25.2.0
6 participants