Deprecation warning for camelCase event names cannot be suppressed about ngvue

[nicolaspayot]

@scottmmjackson yes you're totally right! Thanks for the PR. I'll review it right away.

[jaredmcateer]

Ah I see the problem, I assumed that BABEL_ENV was just some alias/polyfill for variables like NODE_ENV when using Babel, but I actually have to set BABEL_ENV=test in my env variables even though we don't use it.

[scottmmjackson]

Oh, no, I see what's going on.

process.env.BABEL_ENV === 'test' is where it's suppressed. Though, that may be targeting the ngVue tests themselves, and not library consumers?

https://github.com/ngvue/ngvue/blob/a4403837a535a3f26710018712109d7abe622988/lib/logger.js#L2

[jaredmcateer]

How long are you planning on keeping this deprecation warning in ngVue? It is currently spamming the heck out of our tests even though we don't have any camelCase event names.

[scottmmjackson]

@jaredmcateer If you're using babel, set BABEL_ENV=test. Same with webpack, but in a different way.

Yeah, it's a bit jarring, but there's a workaround here.