gh-143181: Fix 'overriden' -> 'overridden' in c-api/module.rst

[duane9]

Doc/c-api/module.rst line 574:

• Issue: Doc: Fix typo in c-api/module.rst - 'overriden' should be 'overridden' #143181

---

📚 Documentation preview 📚: https://cpython-previews--143182.org.readthedocs.build/

[python-cla-bot]

All commit authors signed the Contributor License Agreement.

[hugovk]

There's some "overriden" in test comments, want to fix them too?

Also, no need to create an issue next time for trivial things like typo fixes.

https://devguide.python.org/getting-started/pull-request-lifecycle/#id2

[StanFromIreland]

LGTM

[terryjreedy]

Searching 'overriden' in C:\Programs\Python315\Lib\test\*.py ...
C:\Programs\Python315\Lib\test\test_build_details.py: 36:         """Install details file contents. Should be overriden by subclasses."""
C:\Programs\Python315\Lib\test\test_dict.py: 1584:         # Only TypeError exception is overriden,
C:\Programs\Python315\Lib\test\test_set.py: 664:         # Only TypeError exception is overriden,

Need case-sensitive search to no match OverrideN....
Same in 3.14, not in 3.13. Likely new errors since last general spell check.

[StanFromIreland]

Terry, here is a case-insensitive grep:

$ git grep -ri overriden
Doc/c-api/module.rst:  hook returned (unless overriden with :c:macro:`Py_mod_token`).
Lib/test/test_build_details.py:        """Install details file contents. Should be overriden by subclasses."""
Lib/test/test_descr.py:        class OverrideNeither:
Lib/test/test_descr.py:        self.assertRaises(TypeError, OverrideNeither, 1)
Lib/test/test_descr.py:        self.assertRaises(TypeError, OverrideNeither, kw=1)
Lib/test/test_descr.py:        class OverrideNew:
Lib/test/test_descr.py:        class OverrideBoth(OverrideNew, OverrideInit):
Lib/test/test_descr.py:        for case in OverrideNew, OverrideInit, OverrideBoth:
Lib/test/test_dict.py:        # Only TypeError exception is overriden,
Lib/test/test_set.py:        # Only TypeError exception is overriden,

[duane9]

Changed "overriden" -> "overridden" in the tests. Class names like OverrideNew and OverrideNeither were already spelled correctly with "Override", so they didn't need changing.

[terryjreedy]

My apologies, I meant case-sensitive to avoid fake hits.

[miss-islington-app]

Thanks @duane9 for the PR, and @terryjreedy for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

[miss-islington-app]

Sorry, @duane9 and @terryjreedy, I could not cleanly backport this to 3.14 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 5d1e78f7b59ffa3308755b5b2e0f85eb0c6ac890 3.14

[terryjreedy]

The doc is not in 3.14. As I noted above, the tests are, but they also are not in 3.13.

[terryjreedy]

I tried the 3.14 cherry_picker fix but I don't understand how to fix the merge conflict here. Might be simpler to start fresh. I'll leave for someone else or we can not backport. Not essential here.

[StanFromIreland]

If one were to open a PR that fixes just typos in tests we would generally reject it, I think we should apply the same logic to the backport.