Skip to content

Conversation

@linkmauve
Copy link
Contributor

Thanks to the typos tool!

Previously _PyCoMontoringData was ignored as part of #5064, which is obviously the wrong name. This PR changes it for the real _PyCoMonitoringData name, but since it should be private that shouldn’t affect any real user of the ffi crate.

Thank you for contributing to PyO3!

By submitting these contributions you agree for them to be dual-licensed under PyO3's MIT OR Apache-2.0 license.

Please consider adding the following to your pull request:

  • an entry for this PR in newsfragments - see [https://pyo3.rs/main/contributing.html#documenting-changes]
    • or start the PR title with docs: if this is a docs-only change to skip the check
    • or start the PR title with ci: if this is a ci-only change to skip the check
  • docs to all new functions and / or detail in the guide
  • tests for all new or changed functions

PyO3's CI pipeline will check your pull request, thus make sure you have checked the Contributing.md guidelines. To run most of its tests locally, you can run nox. See nox --list-sessions for a list of supported actions.

Thanks to the typos[1] tool!

Previously _PyCoMontoringData was ignored as part of PyO3#5064[2], which is
obviously the wrong name.  This PR changes it for the real
_PyCoMonitoringData name, but since it should be private that shouldn’t
affect any real user of the ffi crate.

[1] https://github.com/crate-ci/typos
[2] PyO3#5064
@linkmauve linkmauve changed the title Fix typos across the codebase docs: Fix typos across the codebase Oct 30, 2025
Copy link
Member

@davidhewitt davidhewitt left a comment

Choose a reason for hiding this comment

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

Thanks!

@davidhewitt davidhewitt added this pull request to the merge queue Oct 31, 2025
Merged via the queue into PyO3:main with commit d248913 Oct 31, 2025
42 of 44 checks passed
@linkmauve linkmauve deleted the fix-typos branch November 3, 2025 13:08
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.

2 participants