Skip to content

Conversation

@PseudoImplementation
Copy link

No description provided.

@ShadowNinja
Copy link
Member

This breaks recipe alignment.

@kaeza
Copy link
Member

kaeza commented Jul 15, 2014

This breaks recipe alignment.

Assuming you mean in the code, aligment was already broken, and such trivial changes can be done by the person merging this. I've also noticed lots of style issues (unrelated to this change).

TL;DR, style issues shouldn't be cited as reason for "not merging", unless they are particularly egregious.

@VanessaE
Copy link
Contributor

Didn't I already do this once, or rather a similar change some time ago? :-)

(namely in commits 749df3b and dc32481 )

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.

5 participants