search

ehynds / grunt-remove-logging

Grunt.js task for removing console logging statements
MIT License
192 stars 33 forks source link

Opinion : Look into using Esprima, Estraverse, and Escodegen #19

Closed gfranko closed 10 years ago

gfranko commented 10 years ago

It might be cleaner to create and traverse an AST instead of using Regex's. Just a thought! Great job though =)

ehynds commented 10 years ago

Yep. I would actually recommend using grunt-groundskeeper over this one, which was released after this plugin and leverages groundskeeper (which leverages esprima...).

gfranko commented 10 years ago

Nice, thanks!