Skip to content

STY: Fix typos in colormap #22993

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 6, 2022
Merged

STY: Fix typos in colormap #22993

merged 1 commit into from
May 6, 2022

Conversation

matthewfeickert
Copy link
Contributor

PR Summary

PR #22298 got merged with typos before PR #22777, which introduced the codespell pre-commit hook, which would have caught them. This simply fixes those typos. 👍

PR Checklist

Tests and Styling

  • [N/A] Has pytest style unit tests (and pytest passes).
  • Is Flake 8 compliant (install flake8-docstrings and run flake8 --docstring-convention=all).

Documentation

  • [N/A] New features are documented, with examples if plot related.
  • [N/A] New features have an entry in doc/users/next_whats_new/ (follow instructions in README.rst there).
  • [N/A] API changes documented in doc/api/next_api_changes/ (follow instructions in README.rst there).
  • [N/A] Documentation is sphinx and numpydoc compliant (the docs should build without error).

@matthewfeickert
STY: Fix typos in colormap
4ffe491 
PR 22298 got merged with typos before PR 22777, which introduced the
codespell pre-commit hook, which would have caught them. This simply
fixes those typos.
@matthewfeickert
Copy link
Contributor Author

@jklymak given that you approved PR #22777 I'll tag you here for review, though I fully understand that you might not have time to look at it today.

@jklymak
Copy link
Member

jklymak commented May 6, 2022

I'll just merge. We don't env need to run CI on this.

@jklymak jklymak merged commit 9913c81 into matplotlib:main May 6, 2022
@matthewfeickert matthewfeickert deleted the sty/fix-typos-from-22298 branch May 6, 2022 16:46
@matthewfeickert
Copy link
Contributor Author

Thanks!

@QuLogic QuLogic added this to the v3.6.0 milestone May 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jklymak jklymak jklymak approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v3.6.0
Development

Successfully merging this pull request may close these issues.
3 participants
@matthewfeickert @jklymak @QuLogic