mirror of
https://github.com/clockworklabs/SpacetimeDB.git
synced 2026-05-06 23:59:43 -04:00
John Detter
50f5ed034e
# Description of Changes <!-- Please describe your change, mention any related tickets, and so on here. --> This was prompted from another request from the discord. Conversation context: https://discord.com/channels/1037340874172014652/1138987509834059867/1496964407278698656 We've had the ticket to make `--yes` take an enum value: https://github.com/clockworklabs/spacetimedb/issues/3784 . Since we can do this in a non-API/ABI breaking way we're just implementing this ticket. *Disclaimer: I used claude to write ~90% of this PR.* # API and ABI breaking changes <!-- If this is an API or ABI breaking change, please apply the corresponding GitHub label. --> None. `--yes` now just takes an optional argument string that we parse, existing CLI commands that use `--yes` will be unaffected. # Expected complexity level and risk 1 - This is just modifying the `--yes` argument and I've included tests in the PR. <!-- How complicated do you think these changes are? Grade on a scale from 1 to 5, where 1 is a trivial change, and 5 is a deep-reaching and complex change. This complexity rating applies not only to the complexity apparent in the diff, but also to its interactions with existing and future code. If you answered more than a 2, explain what is complex about the PR, and what other components it interacts with in potentially concerning ways. --> # Testing <!-- Describe any testing you've done, and any testing you'd like your reviewers to do, so that you're confident that all the changes work as expected! --> - [x] `spacetime publish --yes` still works as expected - [x] `spacetime publish --yes=all` does the same thing as `spacetime publish --yes` - [x] `spacetime publish --yes=remote` only skips asking if publishing to a remote server is ok. - [x] `spacetime publish --yes=remote` only skips asking if publishing to a remote server is ok. - [x] `spacetime publish --yes=migrate|remote` skips both the migrate prompt and the remote server prompt. - [x] new tests are passing