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

[FIX] Replace all non-breaking spaces with vanilla spaces #536

Merged
merged 3 commits into from
Jul 22, 2020

Conversation

nicholst
Copy link
Collaborator

@nicholst nicholst commented Jul 22, 2020

This PR replaces all non-breaking spaces with spaces (or deletes them if they were adjacent to a space).

As far as I could tell, none of these non-breaking spaces occurred with purpose, but were rather accidental. (Only purpose I can think of is to avoid a number getting separated from a unit, but they have not been used in this way so far in the BIDS spec.)

I realise it doesn't really matter, but my editor highlights them and, in the spirit of keeping clean and simple ascii markdown, I thought they should go.

(For reference these were found with grep $'\xc2\xa0' *.md)

Copy link
Member

@sappelhoff sappelhoff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, I didn't even know there were characters like that. +1 to get rid of them - thanks!

You accidentally misaligned one table pipe, that's why Travis CI is complaining. I made a suggestion to fix it.

src/99-appendices/06-meg-file-formats.md Outdated Show resolved Hide resolved
src/01-introduction.md Outdated Show resolved Hide resolved
src/01-introduction.md Outdated Show resolved Hide resolved
@sappelhoff sappelhoff merged commit 9de5010 into bids-standard:master Jul 22, 2020
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