This is so that the cloud index can add cloud-specific routes?
Aug 9 2021
Aug 3 2021
Jul 31 2021
Jul 30 2021
This makes sense to me... thanks for cleaning this up.
Jul 29 2021
noticed that the future tick colors were also off
just to confirm then... was this not broken on prod?
Jul 28 2021
Yeah, I think the important piece is not being reliant on webpack, so that we can actually do the imports asynchronously from the webworker.
Is there a way to test that the layout worker doesn't have imports (direct or transitive) that will break it?
Jul 27 2021
Jul 23 2021
Jul 22 2021
Jul 20 2021
Jul 19 2021
Jul 16 2021
Jul 15 2021
more mypy, renaming for clarity
In favor of D8385
added comments, renamed some methods, dropped is_enabled
Jul 14 2021
It gets a little hairy if we allow arbitrary dagster event types, because the current implementation (with the RunShardedEventsCursor) implicitly assumes a 1-to-1 event to run mapping.
The run status => event mapping feels a little weird to me, but maybe results in the desired behavior.