noseglid / atom-build

:hammer: Build your project directly from the Atom editor
https://atom.io/packages/build
MIT License
248 stars 97 forks source link

remove ANSI escape sequences #437

Closed fightling closed 8 years ago

fightling commented 8 years ago

remove ANSI escape sequences before matching any regular expression for easy parsing of formatted/coloured build messages.

noseglid commented 8 years ago

This has been discussed multiple times and been decided against. If you want to do error matching you should not output any ansi escape sequences. Atom build does not advertise a pty, so most sane commands would not output any ANSI escape sequences.