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.

Sign up for GitHub

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

Jump to bottom

Separate inline comments by at least two spaces. #488

Merged
merged 1 commit into from
Mar 10, 2018
Merged

Separate inline comments by at least two spaces. #488

merged 1 commit into from
Mar 10, 2018

Conversation

mkcor
Copy link
Contributor

@mkcor mkcor commented Mar 6, 2018

Hello!

For reference: https://www.python.org/dev/peps/pep-0008/#inline-comments

Best,
Marianne

annefou
annefou approved these changes Mar 10, 2018
View reviewed changes
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.

Thanks for your contribution and the link. Maybe we should highlight a bit more how to best use inline comments.

@annefou annefou merged commit bd41119 into swcarpentry:gh-pages Mar 10, 2018
maxim-belkin added a commit that referenced this pull request Oct 23, 2020
@maxim-belkin
GitHub Actions: website (#488)
99ecdf4
zkamvar pushed a commit that referenced this pull request Apr 21, 2023
Merge pull request #488 from mkcor/patch-2
b75239f 
Separate inline comments by at least two spaces.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@annefou annefou annefou approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mkcor @annefou