Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork32k
Fix typo at _interpretersmodule.c file#133174
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
Uh oh!
There was an error while loading.Please reload this page.
Conversation
StanFromIreland commentedApr 30, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
This is pointless and should be closed, it is not user facing and barely a typo. |
Yes, it's not user-facing, but it's CPython-facing! There aren't many experts on |
StanFromIreland commentedApr 30, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Is it really a fix of any importance though? I can read it fine without the space, I would assume most people can too. |
Fixes without priority still have importance. If we want to close this because it's "churn," that's probably fine, but keep in mind that there is real merit to keeping comments in check,especially in parts of the codebase that lack experts. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I agree with Peter -- though I'm not a huge fan of typo-only PRs, this one does seem somewhat beneficial.
A
b99d970
intopython:mainUh oh!
There was an error while loading.Please reload this page.
Fix the typo in the
is_running_main
function comment.