-
-
Notifications
You must be signed in to change notification settings - Fork 155
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added consistent spelling #581
Conversation
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'm concerned by some things, and pointed them below. Anyway, probably it's just me, so if other maintainers disagree, I'm ok with these changes 👍
Shit. Sorry. Trying to get used to this PR thing. Closed the PR by accident. |
Cool. If I need to make a few more changes I'll do that. You guys will just let me know. |
I just edited your first post to link #579 to this PR, so it can be closed when merging this :) |
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.
@CN-M Thanks for working on this, looks great. Have you seen @DiddiLeija 's comment on the Sphinx config?
Co-authored-by: Diego Ramirez <dr01191115@gmail.com>
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.
Now this sounds good to me!
Lovely to hear, @DiddiLeija |
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.
Thank you!
Closes #579
This PR aims to fix the inconsistent spelling of
Nox
andNoxfile
all while leaving thenox
executables andnoxfile.py
files unchanged.