Skip to content

docs: Fix a few typos#94899

Merged
erlend-aasland merged 1 commit into
python:mainfrom
timgates42:bugfix_typos
Aug 8, 2022
Merged

docs: Fix a few typos#94899
erlend-aasland merged 1 commit into
python:mainfrom
timgates42:bugfix_typos

Conversation

@timgates42
Copy link
Copy Markdown
Contributor

There are small typos in:

  • Misc/NEWS.d/3.9.0a1.rst
  • Modules/getpath.py
  • Objects/typeobject.c

Fixes:

  • Should read overridden rather than overriden.
  • Should read calculation rather than calcualation.

Semi-automated pull request generated by
https://github.com/timgates42/meticulous/blob/master/docs/NOTE.md

There are small typos in:
- Misc/NEWS.d/3.9.0a1.rst
- Modules/getpath.py
- Objects/typeobject.c

Fixes:
- Should read `overridden` rather than `overriden`.
- Should read `calculation` rather than `calcualation`.

Signed-off-by: Tim Gates <tim.gates@iress.com>
Copy link
Copy Markdown

@ghagevaibhav ghagevaibhav left a comment

Choose a reason for hiding this comment

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

Changes looks, okay. Thanks for the work. Happy Coding.

@erlend-aasland erlend-aasland added needs backport to 3.10 only security fixes needs backport to 3.11 only security fixes and removed needs backport to 3.10 only security fixes needs backport to 3.11 only security fixes labels Aug 8, 2022
@erlend-aasland
Copy link
Copy Markdown
Contributor

FYI: I reverted my decision regarding backporting these spelling fixups; 3.11 is blocked bco. the upcoming RC1 release; I don't think the PR churn is worth it. I'm fine with applying it to main, though.

@erlend-aasland erlend-aasland merged commit 32b49f6 into python:main Aug 8, 2022
@erlend-aasland
Copy link
Copy Markdown
Contributor

Thanks for the PR, Tim.

iritkatriel pushed a commit to iritkatriel/cpython that referenced this pull request Aug 11, 2022
- overriden => overridden
- calcualation => calculation

Signed-off-by: Tim Gates <tim.gates@iress.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants