NotificationsYou must be signed in to change notification settings
Fork7.9k
Star21.5k

Fix several minor typos#30281

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

timhoffm merged 1 commit intomatplotlib:mainfromQuLogic:typo

Jul 10, 2025

Merged

Fix several minor typos#30281

timhoffm merged 1 commit intomatplotlib:mainfromQuLogic:typo

Jul 10, 2025

Conversation

Copy link

Member

QuLogic commentedJul 10, 2025

PR summary

Just a few new things found with codespell.

Also, don't worry about regenerating the issue/PR lists; I've fixed all titles on GitHub already.

PR checklist

[n/a] "closes #0000" is in the body of the PR description tolink the related issue
[n/a] new and changed code istested
[n/a]Plotting related features are demonstrated in anexample
[n/a]New Features andAPI Changes are noted with adirective and release note
Documentation complies withgeneral anddocstring guidelines

Fix several minor typos

7432ef4

github-actionsbot added topic: axes topic: tri Documentation: user guide

files in galleries/users_explain or doc/users

Documentation: APIfiles in lib/ and doc/api labels

Jul 10, 2025

QuLogic added this to thev3.11.0 milestone

Jul 10, 2025

QuLogic added the Maintenance label

Jul 10, 2025

oscargus approved these changes

Jul 10, 2025

View reviewed changes

Copy link

Member

oscargus left a comment

Choose a reason for hiding this comment

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

One may argue if PR titles should be corrected as that was the name of the PR merged, but apart from that looks good. (And not sure I will argue.)

timhoffm approved these changes

Jul 10, 2025

View reviewed changes

Copy link

Member

timhoffm commentedJul 10, 2025

One may argue if PR titles should be corrected as that was the name of the PR merged, but apart from that looks good. (And not sure I will argue.)

We have rejected this before in other PR's, but since@QuLogic has taken the effort to update the PR titles in GitHub, that's fine. Not sure whether I'd have spent the time, but we're now in a nice and clean state.