Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

Added semicolons so the linter won't scream anymore #20

Closed
wants to merge 2 commits into from

Conversation

thanpolas
Copy link

I suspect, omitting semicolons was by choice, but i have to try, considering i intent on pushing more code and my live linter went bananas =)

@ghost
Copy link

ghost commented Aug 26, 2012

Tell your linter he's too old for this shit

@thanpolas thanpolas closed this Aug 26, 2012
@thanpolas
Copy link
Author

what would life be without trolls

@medikoo
Copy link

medikoo commented Aug 27, 2012

@thanpolas When committing to open source we should rather obey code style conventions chosen by authors and not force ours.
I personally also use semicolons, but it was already proven that we can quite safely live without them, and it's nice to respect others choices on that :)

@thanpolas
Copy link
Author

@medikoo there was no style guide on the docs and i didn't realize it was a style. As soon as i did, i removed the commit.

Take it easy everyone, we are all on the same side

@ghost
Copy link

ghost commented Aug 27, 2012

it feels like a bad joke, but if we all on the same side the boat will soon flip over

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.

2 participants