jamiebrynes7 commented 4 years ago

This PR contains (most of) the upgrade to the Worker SDK 14.1.0.

It notably is missing:

The Schema_Bundle and Schema_Json APIs
JSON snapshot support

The first of these is implemented, but I couldn't quite get it to work, so I figured lets get the rest of this in.

codecov[bot] commented 4 years ago

Codecov Report

Merging #157 into master will decrease coverage by 0.05%. The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #157      +/-   ##
==========================================
- Coverage   14.08%   14.03%   -0.06%     
==========================================
  Files          38       39       +1     
  Lines        3790     3806      +16     
==========================================
  Hits          534      534              
- Misses       3256     3272      +16

Impacted Files	Coverage Δ
spatialos-sdk/src/worker/schema.rs	`3.81% <ø> (ø)`	:arrow_up:
spatialos-sdk-sys/src/worker.rs	`0% <ø> (ø)`	:arrow_up:
spatialos-sdk/src/worker/schema/generic_data.rs	`0% <0%> (ø)`
cargo-spatial/src/config.rs	`0% <0%> (ø)`	:arrow_up:
project-example/src/connection_handler.rs	`0% <0%> (ø)`	:arrow_up:
spatialos-sdk/src/worker/connection.rs	`0% <0%> (ø)`	:arrow_up:
spatialos-sdk/src/worker/parameters.rs	`4.85% <0%> (-0.08%)`	:arrow_down:

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 73c971b...f2ee03c. Read the comment docs.

dgavedissian commented 4 years ago

out of interest, why not upgrade to 14.4.0?

jamiebrynes7 commented 4 years ago

out of interest, why not upgrade to 14.4.0?

Doing it in steps to make both the PRs more reviewable and the work easier to chunk. The plan is to bring it all the way up to latest :smile:

jamiebrynes7 / spatialos-sdk-rs

Upgrade to Worker SDK 14.1.0 #157

Codecov Report