Skip to content
This repository was archived by the owner on Nov 9, 2017. It is now read-only.

Conversation

@robrix
Copy link

@robrix robrix commented Aug 25, 2014

Brings in the no trailing whitespace section from github/objective-c-style-guide#58.

Copy link
Contributor

Choose a reason for hiding this comment

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

/nope

Copy link
Contributor

Choose a reason for hiding this comment

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

I normally try to be understanding of your multiline spacing tendencies, but this is just input into a rendered document. The blank line doesn't do anything, and the header syntax (####) separates things well enough on its own.

Choose a reason for hiding this comment

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

I was wondering about this too, @robrix

Copy link
Author

Choose a reason for hiding this comment

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

Newline + #### makes it much easier for me to spot and parse the sections than newline alone.

I accept that this may not be a problem for you. Does this formatting introduce issues (technical or otherwise) for you?

Choose a reason for hiding this comment

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

Does this formatting introduce issues (technical or otherwise) for you?

No, I'm personally just curious.

Copy link
Author

Choose a reason for hiding this comment

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

That was primarily directed at @jspahrsummers 😊

Copy link
Contributor

Choose a reason for hiding this comment

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

@jspahrsummers jspahrsummers assigned jssjr and jspahrsummers and unassigned jssjr Aug 25, 2014
@jspahrsummers
Copy link
Contributor

:godmode:

jspahrsummers added a commit that referenced this pull request Aug 25, 2014
Don’t leave trailing whitespace section
@jspahrsummers jspahrsummers merged commit e0571a5 into master Aug 25, 2014
@jspahrsummers jspahrsummers deleted the no-trailing-whitespace branch August 25, 2014 22:56
@jspahrsummers jspahrsummers removed their assignment May 22, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants