-
Notifications
You must be signed in to change notification settings - Fork 10
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
Added some quotes! #41
base: master
Are you sure you want to change the base?
Conversation
|
||
- :c: Hope That Firewall works because your screwed... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd vote this to be left out, because it doesn't seem anything special. @moll agree?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Concur. Not much of an aphorism. >)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll rectify it all today! Let me know about the white spaces problem, I'm
not getting where exactly white spaces are there! Are you talking about the
extra line? After each quote?
On 18-Jan-2016 12:10 am, "Andri Möll" [email protected] wrote:
In quotes.yaml
#41 (comment):+- :c: Hope That Firewall works because your screwed...
Concur. Not much of an aphorism. >)
—
Reply to this email directly or view it on GitHub
https://github.com/jarmo/hackersays/pull/41/files#r49950275.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think he's talking about trailing whitespace on line 32.
Added some inline comments. I'm also pretty sure that you've added some quotes there, which already exists in |
No description provided.