[performance] only logLibraryVersions in debug mode

[GavinJoyce]

TODO:

• verify that the ember inspector continues to work: https://github.com/emberjs/ember-inspector/blob/679643037032fab3e4388a8dcb8cd0f20d1cc660/shared/in-page-script.js#L15

[rwjblue]

We should also remove the invocation here by wrapping in runInDebug so the whole function gets DCE by uglify.

[locks]

do we need this with the no-opping body?

[GavinJoyce]

See Robert's comment above

…

On 12 Jan 2017 5:37 p.m., "Ricardo Mendes" ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In packages/ember-application/lib/system/application.js <#14829 (review)> : > @@ -339,7 +340,7 @@ const Application = Engine.extend({ } registerLibraries(); - logLibraryVersions(); + runInDebug(() => logLibraryVersions()); do we need this with the no-opping body? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#14829 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAJ3j3bAs_QlTHd4C8ZzKxa4UqKDez2ks5rRmTWgaJpZM4Lh-eI> .

[rwjblue]

LGTM, will land once CI is finished.

[GavinJoyce]

🍏 🚢