Remove mobx-devtools-mst

[dependabot]

Bumps mobx-devtools-mst from 0.9.30 to 0.9.34.

Commits

• See full diff in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

---

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

• @dependabot rebase will rebase this PR
• @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
• @dependabot merge will merge this PR after your CI passes on it
• @dependabot squash and merge will squash and merge this PR after your CI passes on it
• @dependabot cancel merge will cancel a previously requested merge and block automerging
• @dependabot reopen will reopen this PR if it is closed
• @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
• @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
• @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

[coveralls]

coverage: 77.827% (+0.03%) from 77.793%
when pulling 055949d on dependabot/npm_and_yarn/mobx-devtools-mst-0.9.34
into 1d48e37 on master.

[shaunanoordin]

Why is this small bump triggering an error?

unhandledRejection ReferenceError: self is not defined
    at Object.<anonymous> (/home/runner/work/front-end-monorepo/front-end-monorepo/node_modules/mobx-devtools-mst/lib/index.js:1:342)

Let me re-run the tests.

[shaunanoordin]

Tests are still failing upon a rerun. 🤨 This seems like it's going to be a thing, unfortunately.

[goplayoutside3]

Weirdly, I can't replicate the error locally at all. Even if I run yarn workspace @zooniverse/fe-project build with env set to production like the CI build action, it works fine.

[goplayoutside3]

But also this dependabot PR edits the yarn.lock to mobx-devtools-mst to v0.9.34, yet package.json for app-project and lib-classifier specify ~0.9.21 🤔 Edit: this does actually make sense.

Might be using this dependabot PR to simply remove the devtools if no one on the zoo team uses them.

[goplayoutside3]

I suspect the error Shaun mentioned above is due to makeInspectable() being called on the server in _app.js. None of the dev team uses mobx-devtools-mst and there's lack of documentation from the most recent releases. Removing.