mobx-keystone
mobx-keystone copied to clipboard
update compiler configs and docs
It seems using different compiler settings for MobX 4 and 5/6 isn't necessary. Do you think this is generally valid? At least the test suite passes.
βοΈ Deploy Preview for mobx-keystone ready!
π¨ Explore the source changes: d5faaba95f5d7268ffb5b568eba9a12c530d30f6
π Inspect the deploy log: https://app.netlify.com/sites/mobx-keystone/deploys/6220da16abd31100077013b7
π Browse the preview: https://deploy-preview-421--mobx-keystone.netlify.app
Codecov Report
Merging #421 (d5faaba) into master (17ffa8e) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #421 +/- ##
=======================================
Coverage 90.93% 90.93%
=======================================
Files 179 179
Lines 6388 6388
Branches 1168 1157 -11
=======================================
Hits 5809 5809
Misses 542 542
Partials 37 37
Continue to review full report at Codecov.
Legend - Click here to learn more
Ξ = absolute <relative> (impact)
,ΓΈ = not affected
,? = missing data
Powered by Codecov. Last update 17ffa8e...d5faaba. Read the comment docs.
I don't know, I think those config were taken directly from mobx docs, so while they might work with keystone they might not work if you use decorators for mobx itself
Okay, I'll investigate a bit further.