Use JS macrotask executor for CE 2

[danicheg]

Even if we don't use fatal warnings, it, all the same, strikes the eye on every compile, CI run.

The global execution context in Scala.js is based on JS Promises (microtasks).
[warn] Using it may prevent macrotasks (I/O, timers, UI rendering) from running reliably.
[warn] 
[warn] Unfortunately, there is no way with ECMAScript only to implement a performant
[warn] macrotask execution context (and hence Scala.js core does not contain one).
[warn] 
[warn] We recommend you use: https://github.com/scala-js/scala-js-macrotask-executor
[warn] Please refer to the README.md of that project for more details regarding
[warn] microtask vs. macrotask execution contexts.
[warn] 
[warn] If you do not care about macrotask fairness, you can silence this warning by:
[warn] - Adding @nowarn("cat=other") (Scala >= 2.13.x only)
[warn] - Setting the -P:scalajs:nowarnGlobalExecutionContext compiler option
[warn] - Using scala.scalajs.concurrent.JSExecutionContext.queue
[warn]   (the implementation of ExecutionContext.global in Scala.js) directly.
[warn] 
[warn] If you do not care about performance, you can use
[warn] scala.scalajs.concurrent.QueueExecutionContext.timeouts().
[warn] It is based on setTimeout which makes it fair but slow (due to clamping)
[warn]                 
[warn]     IO.contextShift(ExecutionContext.global)

[armanbilge]

Hmm, why not use the MacrotaskExecutor like the warning suggests?

[danicheg]

@armanbilge because it brings new dependency and needs full-fledged testing. And yeah, I'm not an actual ScalaJS user.

[danicheg]

Damn it, Scala 3 doesn't contain -P:scalajs:nowarnGlobalExecutionContext option :(

[armanbilge]

FWIW the MacrotaskExecutor project was extracted from CE, is maintained by Daniel and I, and is the default in CE3.

[danicheg]

@armanbilge so, I've followed your (and compiler's one) advice to use scala-js-macrotask-executor for CE2. If CE3 uses it as a default, I think it'd be fine for CE2 as well.

[armanbilge]

Is it possible to use munitExecutionContext instead?

[danicheg]

That was intentional #65

[armanbilge]

Wow, munit uses parisitic EC ?? thanks for the pointer.

[armanbilge]

Thanks, LGTM 👍