Skip to content
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

Change variable 'mySalsa' to 'my_salsa' for PEP 8 compliance. #484

Merged
merged 1 commit into from
Mar 10, 2018

Conversation

jason-ellis
Copy link
Contributor

No description provided.

Copy link

@annefou annefou left a comment

Choose a reason for hiding this comment

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

Yes. looking through the lesson the prevailing style for naming variable seems to be using underscore so we should stick to it.

Thanks for your contribution.

@annefou annefou merged commit 6a09fee into swcarpentry:gh-pages Mar 10, 2018
fmichonneau pushed a commit to fmichonneau/python-novice-inflammation that referenced this pull request Oct 13, 2020
maxim-belkin added a commit that referenced this pull request Oct 23, 2020
zkamvar pushed a commit that referenced this pull request Apr 21, 2023
Change variable 'mySalsa' to 'my_salsa' for PEP 8 compliance.
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