Skip to content

Conversation

@dakixr
Copy link
Contributor

@dakixr dakixr commented Aug 4, 2025

Problem at hand:
Screenshot 2025-08-04 at 11 31 52

After PR:
Screenshot 2025-08-04 at 11 32 48
Screenshot 2025-08-04 at 11 34 16

@jodal
Copy link
Contributor

jodal commented Aug 4, 2025

I've noticed this problem as well. Would be nice to get it fixed.

Can we avoid duplicating the long docstring, e.g. by removing it from the class?

Please remove the existing update_wrapper call on line 52, since it is obviously not effective.

@pelme
Copy link
Owner

pelme commented Aug 5, 2025

Good catch, LGTM with @jodal's suggestions.

@dakixr
Copy link
Contributor Author

dakixr commented Aug 5, 2025

Thanks for the feedback!

@pelme pelme merged commit 66bd301 into pelme:main Aug 5, 2025
11 checks passed
@jodal
Copy link
Contributor

jodal commented Aug 15, 2025

@pelme Could we get this released? :-)

@pelme
Copy link
Owner

pelme commented Aug 15, 2025

sorry about the delay! we have such an amazing workflow for cutting new releases now so it is a breeze 😆

@pelme
Copy link
Owner

pelme commented Aug 15, 2025

25.8.1 is out now

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.

3 participants