Conversation
|
Not sure why we are doing this. I am probably one of the people who used double spaces here. I think they help separate sentences from each other within a paragraph when in a monospace environment, for example in an editor like vscode. And the end result on a web page is identical. |
|
I mean it would show up as a mistake in spell checks I think. I really haven't seen double spaces used in any "code" (or text) anywhere, so I think this is the better way (nowadays). Furthermore, Nonetheless, this is obviously a very minor change, so whatever... |
|
I have never seen any spell checker complain about this. On the other hand, if everybody here thinks that should be the standard, we should add it to the pi-base wiki as a style guideline, and I will follow it in the future. Otherwise, not. @StevenClontz we need your input here. |
|
I don't think we should need a policy here one way or the other for now. If we want one, I'd ask for three things:
With respect to all parties, I'm going to close this PR for now, but anyone is welcome to reopen if they feel strongly that conversation should continue. (Personally, I am Team Single-Space -- it should be the job of the software, not the author, to make things look "good" in the software interface. But if we had all of the above set up, I wouldn't care which one decide is "right", unless I guess someone can prove that double-spacing adds a significant amount of extra size to our data blobs.) |
|
Thanks Steven. Personally, I think 1, 2, 3 is overkill. If everyone here is in Team Single-Space, there shouldn't be a problem in adding a "recommendation" in the pi-base wiki. And I would follow that recommendation. So, if anyone strongly cares, please follow up and get consensus on it. Otherwise, it will remain as is. |
Changes
.to.Arguably not the most important PR, but this improves the style slightly.
I went over the changes (briefly) and it looks good.