Skip to content

Conversation

brendan-kellam
Copy link
Contributor

Updates the demo deploy cadence to be on tag instead of main. The reasoning is that I would like to make breaking REST API changes (like in #555) without having to spin a new release. However, MCP depends on the demo instance's REST api not breaking (on non-version changes), so that complicates things.

Whether we should worry about breaking public MCP is a separate discussion. For now, let's avoid breaking it.

@brendan-kellam brendan-kellam merged commit 18ba1d2 into main Oct 8, 2025
7 checks passed
Copy link

github-actions bot commented Oct 8, 2025

@brendan-kellam your pull request is missing a changelog!

Copy link

coderabbitai bot commented Oct 8, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bkellam/demo_deploy_cadence

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@brendan-kellam brendan-kellam deleted the bkellam/demo_deploy_cadence branch October 8, 2025 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant