trace(): log when computed becomes suspended

[urugator]

trace logs when computed becomes suspended as discussed in:
#2859 (comment)

requiresObserver no longer throws, but warns, so it's consistent with computedRequiresReaction, observableRequiresReaction, etc

Improved console messages.

[changeset-bot]

🦋 Changeset detected

Latest commit: 9eca68b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
mobx	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[mweststrate]

I didn't see this change in the PR?

[urugator]

typo ... requiresReaction ... the condition with die is deleted and merged to condition with warn:

 warnAboutUntrackedRead_() {
        if (!__DEV__) return
---     if (this.requiresReaction_ === true) {
---             die(`[mobx] Computed value ${this.name_} is read outside a reactive context`)
---     }
        if (this.isTracing_ !== TraceMode.NONE) {
            console.log(
                `[mobx.trace] '${this.name_}' is being read outside a reactive context. Doing a full recompute`
                `[mobx.trace] Computed value '${this.name_}' is being read outside a reactive context. Doing a full recompute.`
            )
        }
---     if (globalState.computedRequiresReaction) {
+++     if (globalState.computedRequiresReaction || this.requiresReaction_) {
            console.warn(
---            `[mobx] Computed value ${this.name_} is being read outside a reactive context. Doing a full recompute`
+++            `[mobx] Computed value '${this.name_}' is being read outside a reactive context. Doing a full recompute.`
            )
        }
    }

[mweststrate]

Looking good, thanks!