Skip to content

config/services: use parens instead of hyphens#609

Merged
ericonr merged 1 commit into
void-linux:masterfrom
Goorzhel:patch-1
Oct 30, 2021
Merged

config/services: use parens instead of hyphens#609
ericonr merged 1 commit into
void-linux:masterfrom
Goorzhel:patch-1

Conversation

@Goorzhel

Copy link
Copy Markdown
Contributor

A line-initial hyphen makes the line a bulleted list.

@Vaelatern

Copy link
Copy Markdown
Member

Good catch, not sold on the fix.

@Goorzhel

Copy link
Copy Markdown
Contributor Author

Why not? Is it best to stick to ASCII? If so, I suppose an alternate course would be:

services for ttys 1 to 6. This way, package updates which affect these services -
in this case, the `runit-void` package - won't re-enable them.

@Vaelatern

Copy link
Copy Markdown
Member

reasonable immediate fix, yes.

@ericonr

ericonr commented Oct 25, 2021

Copy link
Copy Markdown
Member

That would break vmdfmt.

@ericonr

ericonr commented Oct 25, 2021

Copy link
Copy Markdown
Member

lowdown doesn't create a list here, I guess the bug is with cmark (or lowdown isn't following the spec).

@ericonr

ericonr commented Oct 25, 2021

Copy link
Copy Markdown
Member

@Vaelatern an em-dash is the correct punctuation to use in that spot, the main issue is that it isn't trivial to input, and should be used all across docs if we switch.

@Goorzhel

Copy link
Copy Markdown
Contributor Author
services for ttys 1 to 6. This way, package updates which affect these services
(in this case, the `runit-void` package) won't re-enable them.

A line-initial hyphen makes the line a bulleted list.
@Goorzhel Goorzhel changed the title config/services: use em-dashes instead of hyphens config/services: use parens instead of hyphens Oct 27, 2021
Comment thread src/config/services/index.md
@ericonr ericonr merged commit 184ca5c into void-linux:master Oct 30, 2021
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.

3 participants