Skip to content

Check for messages length before logging.#257

Merged
RyanZim merged 2 commits intopostcss:masterfrom
XhmikosR:master
Jan 4, 2019
Merged

Check for messages length before logging.#257
RyanZim merged 2 commits intopostcss:masterfrom
XhmikosR:master

Conversation

@XhmikosR
Copy link
Copy Markdown
Contributor

@XhmikosR XhmikosR commented Jan 4, 2019

Fixes #256

It doesn't seem to be tests for this, so I hope this doesn't break anything inadvertently.

/CC @RyanZim

@coveralls
Copy link
Copy Markdown

coveralls commented Jan 4, 2019

Coverage Status

Coverage decreased (-0.4%) to 75.342% when pulling 74cc874 on XhmikosR:master into afdc65f on postcss:master.

@postcss postcss deleted a comment from coveralls Jan 4, 2019
@RyanZim RyanZim added the bug label Jan 4, 2019
@XhmikosR
Copy link
Copy Markdown
Contributor Author

XhmikosR commented Jan 4, 2019

@RyanZim: please check the second patch and let me know.

@RyanZim RyanZim merged commit 8c0a465 into postcss:master Jan 4, 2019
@RyanZim
Copy link
Copy Markdown
Collaborator

RyanZim commented Jan 4, 2019

Published in v5.1.1 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants