Skip to content
This repository was archived by the owner on Dec 19, 2024. It is now read-only.

Found a way to emit messages (error/warning) #64

Closed
MoOx opened this issue Jan 30, 2015 · 4 comments
Closed

Found a way to emit messages (error/warning) #64

MoOx opened this issue Jan 30, 2015 · 4 comments
Assignees

Comments

@MoOx
Copy link
Owner

MoOx commented Jan 30, 2015

Currently some postcss plugins report some errors/warning using console.

When you got something like webpack-dev-server, it's just horrible to debug.
Sometimes you don't even see those messages, not sure why.

We need to add to all plugins that throw messages the ability to adjust what to use to handle errors & warnings.

We can

  • throw some errors in plugins then handle the exceptions, but some people where not happy with that
  • provide a callback to handle messages like webpack does: emitWarning & emitError (default to console.warn & console.error)

Ideas?

@MoOx
Copy link
Owner Author

MoOx commented Feb 6, 2015

Since this "feature" will not bring any breaking change, I am going to release 1.0 without that because I think it's better to bring new APIs first in PostCSS.
Here is a issue for that postcss/postcss#177

@MoOx MoOx removed this from the v1.0.0 milestone Feb 6, 2015
@MoOx
Copy link
Owner Author

MoOx commented Apr 2, 2015

postcsss 4.1 is out with the requested API. We "just need" to update all plugin that output warning and stuff.

@magsout
Copy link
Contributor

magsout commented Apr 2, 2015

Let's GO

@MoOx
Copy link
Owner Author

MoOx commented Jun 4, 2015

Closing in favor of #88

@MoOx MoOx closed this as completed Jun 4, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants