-
Notifications
You must be signed in to change notification settings - Fork 30
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
Unwanted 1-space indent #41
Comments
Hey! It's funny, that this package exists for a few years now and you are the first one to notice that 1-space thing. Frankly, I was hesitant to change this, since pretty much everybody using the package will see the diff upon their next update. But, hey, let's keep things tidy =) On the optional line note, I don't really want to introduce it. Making an unconditional switch seems awkward and adding an option seems a bit of an overkill. Hopefully, it's not that big of a deal to you =) |
Guess I'll survive :-) thing is I wasn't able to come with an idea that doesn't involve re-writing the function. |
You will only need to add one line there. But, yeah, I don't think you can do this without re-defining it, at least, for now. |
Alright thanks 👍
Le jeu. 18 mai 2017 14:18, Sergei Nosov <notifications@github.com> a écrit :
You will only need to add one line there. But, yeah, I don't think you can
do this without re-defining it, at least, for now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#41 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AFWUsEfArUAcwOk7hpX2jCGTxcHMV3ATks5r7Db7gaJpZM4Neel7>
.
--
Mathieu Marques
|
Although 2. should be optional I suppose.
Should be:
The text was updated successfully, but these errors were encountered: