@ember/test-waiters getting required in production app?

[jacobq]

I think the reason that this slipped through CI is that during testing we have the devDependencies installed so the module gets resolved. However, when I tried to use this (v0.6.0) in a production app I run into an error like this:

ember-cli-plotly tried to import "@ember/test-waiters" but the package was not resolvable from C:\Users\user\path_to\projects\node_modules\ember-cli-plotly

@Gamma169 or @lsmun, did you encounter this problem?

[jacobq]

I am pretty sure this is coming from 68b1954#diff-a3c0481f819ddcb17ceef5ebe509b39173047a05fd644b05c8f82d4180527961R5
but I'm not sure if I should fix by moving @ember/test-waiters from devDeps to deps or some other way (can't conditionally import AFAIK)

[Gamma169]

@jacobq, Hi we haven't pushed the new version into our production app yet. We needed the update for a new project and was planning on updating our production app in the new year. So we haven't come across this yet.

Since this module is integral to our production application, I will set aside time in the new year to help resolve issues that have come up and the things you mentioned in the last issue.
#176 (comment)

In the meantime, hope you have a nice holiday season! :)