-
Notifications
You must be signed in to change notification settings - Fork 3k
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
MUST KILL NO SPACE BRACES #1868
Comments
I'm assigning myself so no one else takes this one up. But it's all you, @ladyleet. |
Add spaces to import statement on line 44 as per issue ReactiveX#1868.
lol I thought that was how you liked it since I saw it done everywhere. |
@jayphelps you're way more interesting on github than twitter. i'm kind of digging it. |
@ladyleet github is life |
Add spaces to import statement on line 44 as per issue #1868.
I love |
NO @staltz! I'll accept your Finnish Notation, but you can't have no-space-braces! You can't! 😤 |
Yeah, I understand you americans. Inches, Fahrenheit, Feet, AM/PM, Miles. And now... spaces for braces. I guess if you use wrong standards, you gotta consciously and consistently use wrong standards everywhere. :)
|
@staltz: Trump's nukes won't care if they miss by inches or centimeters. The end is nigh. |
But heating something up to 100 degrees (Celsius? Fahrenheit?) might mean life or death. |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
This is a special issue for @ladyleet, who wants to contribute in a meaningful way.
I HATE no space imports in our files. I hate them. We all hate them. @jayphelps hates them. @kwonoj hates them. There's so much hate. End our hate, Tracy. End it.
You wanted to contribute, @ladyleet, here it is... kill those no-space imports!
GOOD:
BAD:
related #1867
The text was updated successfully, but these errors were encountered: