Skip to content

Conversation

@petterreinholdtsen
Copy link
Collaborator

@petterreinholdtsen petterreinholdtsen commented Jul 17, 2022

Use double quotes consistently instead of a mix of single and double quotes. Picked double over single quotes as it had most existing use.

@silopolis
Copy link
Contributor

silopolis commented Jul 17, 2022 via email

@petterreinholdtsen
Copy link
Collaborator Author

petterreinholdtsen commented Jul 17, 2022 via email

@SebKuzminsky
Copy link
Collaborator

What is the significance of switching from single-quotes (') to double-quotes (") here? In other words, why is that part of this patch useful?

@silopolis
Copy link
Contributor

silopolis commented Jul 18, 2022 via email

@petterreinholdtsen
Copy link
Collaborator Author

And once we know which quotes to use, it could get included in the style guide in #1530.

@silopolis
Copy link
Contributor

silopolis commented Jul 20, 2022 via email

@petterreinholdtsen
Copy link
Collaborator Author

petterreinholdtsen commented Jul 20, 2022 via email

@petterreinholdtsen petterreinholdtsen changed the title Improved hostmot2(9) language and made its formatting more consistent Use double quotes consistently in hostmot2(9) Jul 20, 2022
Use double quotes consistently instead of a mix of single and double
quotes.  Picked double over single quotes as it had most existing use.
@petterreinholdtsen
Copy link
Collaborator Author

petterreinholdtsen commented Jul 25, 2022 via email

@silopolis
Copy link
Contributor

silopolis commented Jul 25, 2022 via email

@jepler jepler merged commit 4b608a9 into LinuxCNC:master Jul 28, 2022
@petterreinholdtsen petterreinholdtsen deleted the doc-lang-hostmot2 branch August 7, 2022 16:37
@petterreinholdtsen
Copy link
Collaborator Author

petterreinholdtsen commented Oct 11, 2022 via email

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.

4 participants