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

Update 06-style-guide.md #181

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update 06-style-guide.md #181

wants to merge 1 commit into from

Conversation

cleary3
Copy link

@cleary3 cleary3 commented Feb 26, 2018

Adding figure dash, en dash, and em dash. Per swcarpentry/python-novice-inflammation#478 (comment)

@rgaiacs
Copy link
Contributor

rgaiacs commented Feb 27, 2018

@cleary3 Thanks for the pull request. This need its own section. http://swcarpentry.github.io/lesson-example/06-style-guide/index.html should have four sections, for exemple

Keyboard Key

...

Keyboard Key Combination

...

Links

...

Dahses

On the lessons we make use of figure dash, en dash, and em dash following the ? style guide. Figure dash should be used for ... In the Markdown file, please use ... to insert the figure dash. En dash should be used for ... In the Markdown file, please use ... to insert the en dash. Em dash should be used for ... In the Markdown file, please use ... to insert the em dash.

My recommendation is require authors to use – and — because it make easy to find misuse of one dash.

Said that, I'm going to add this for discussion with other mentors. Thanks again for the pull request.

@cleary3
Copy link
Author

cleary3 commented Feb 27, 2018

Ah ha, now the requested change is clearer to me. Thanks!

Since you're wanting more discussion, I'll leave well enough alone for now. If you decided you do want the new section about Dashes, let me know and I can update the pull request.

@rgaiacs rgaiacs added this to the v9.4.1 milestone Apr 17, 2018
@rgaiacs rgaiacs modified the milestones: v9.5.0, v9.6.0 May 24, 2018
maxim-belkin pushed a commit to maxim-belkin/swc-lesson-example that referenced this pull request Jun 19, 2018
@fmichonneau fmichonneau changed the base branch from gh-pages to main October 13, 2021 06:10
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