search

clockworklabs / SpacetimeDB

Multiplayer at the speed of light
https://spacetimedb.com
Other
4.41k stars 110 forks source link

Fix declaration order of ScheduleAt in C# #2007

Open kazimuth opened 14 hours ago

kazimuth commented 14 hours ago

And add some tests to sdk-test-cs. This was causing ScheduleAt to not get recognized by the schema validation code.

These tests don't fully exercise scheduling/indexes yet, but at least they check if they compile & are accepted by the host.

Expected complexity level and risk

0

Testing

Added tests.

RReverser commented 17 minutes ago

Hm this doesn't look right. It seems it's Rust that is wrong here as C# implemented the enum according to the spec: https://github.com/clockworklabs/SpacetimeDBPrivate/blob/592a34c0cb82d587843d2b2309b824e94c035d26/proposals/0005-timers.md?plain=1#L82

Or, I guess, we can do this PR, but then we need to change the spec as well.