Skip to content

Move to option 'ignoreWhitespaces'#70

Merged
eGavr merged 1 commit intomasterfrom
issue69
Jul 24, 2014
Merged

Move to option 'ignoreWhitespaces'#70
eGavr merged 1 commit intomasterfrom
issue69

Conversation

@eGavr
Copy link
Copy Markdown
Member

@eGavr eGavr commented Jul 24, 2014

closes #69

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-2.04%) when pulling 892f8c9 on issue69 into c3cfc51 on master.

Comment thread lib/index.js
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Варнинги обычно красят жёлтым, а ошибки красным, но не сильно принципиально.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не! красное прям сразу в глаза бросается! пусть останется!

@blond
Copy link
Copy Markdown
Member

blond commented Jul 24, 2014

В README.md, наверное, тоже надо обновить.

@eGavr
Copy link
Copy Markdown
Member Author

eGavr commented Jul 24, 2014

Реадми уже отдельным пул реквесто висит! я все сделаю!

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-2.67%) when pulling 4b74750 on issue69 into c3cfc51 on master.

@blond
Copy link
Copy Markdown
Member

blond commented Jul 24, 2014

:squirrel:

@eGavr
Copy link
Copy Markdown
Member Author

eGavr commented Jul 24, 2014

аееее) белка)

eGavr added a commit that referenced this pull request Jul 24, 2014
Move to option 'ignoreWhitespaces'
@eGavr eGavr merged commit c5811b4 into master Jul 24, 2014
@eGavr eGavr deleted the issue69 branch July 24, 2014 20:26
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.

Move to option 'ignoreWhitespaces' instead of 'ignoreWhitespace'

4 participants