MNT: upgrade codespell (v2.3.0 -> v2.4.1)#17720
Conversation
|
Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.
|
|
👋 Thank you for your draft pull request! Do you know that you can use |
v2.3.0 -> v2.4.1)
|
I don't see the point of this tool renaming a variable |
There's not point, it's purely a side effect from Since @eerovaher opened #17725 with an even more opinionated approach, I'm going to revamp this one to simply ignore this one word so hopefully one of these PRs can be merged quickly. |
78a0e56 to
3f21a4a
Compare
| >>> cli = SAMPIntegratedClient() | ||
| >>> ... | ||
| >>> cli.ereply("abd", SAMP_STATUS_ERROR, result={}, | ||
| >>> cli.ereply("message_id", SAMP_STATUS_ERROR, result={}, |
pllim
left a comment
There was a problem hiding this comment.
Much less controversial now. Thanks!
Description
This is a semi manual upgrade (originally automated in #17710).
All changes are automatic, except for a couple
# codespell:ignorecomments that I added manually.The update is a bit noisy because of
indxvariables being re-namedindex, though I checked that it only happened in private scopes so it shouldn't have any user-visible effect.