Skip to content
This repository was archived by the owner on May 11, 2026. It is now read-only.

New branch build#16

Open
georgevanderson wants to merge 6 commits into
mainfrom
new-branch-build
Open

New branch build#16
georgevanderson wants to merge 6 commits into
mainfrom
new-branch-build

Conversation

@georgevanderson
Copy link
Copy Markdown

No description provided.

@fiveonefour-hosting
Copy link
Copy Markdown

fiveonefour-hosting Bot commented Apr 17, 2026

The latest updates on your projects. Learn more about Boreal.

Project Deployment Logs Base URL Updated (UTC)
planes 🏗️ Building Logs https://514-demos-planes-new-branc-12acb.boreal.cloud 2026-04-17 03:44:25

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
planes Ready Ready Preview, Comment Apr 17, 2026 3:44am

Request Review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b241563281

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread moose/moose.config.toml
workflows = true
data_model_v2 = true
ddl_plan = true
migrate_with_deltas = true
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Restore the supported ddl_plan feature flag

This change replaces ddl_plan with migrate_with_deltas under [features], but Moose feature configuration documents ddl_plan (and not migrate_with_deltas) as the migration/DDL gating flag. As a result, the previous behavior is no longer guaranteed: the setting can be ignored (falling back to defaults) or rejected by strict parsing, which changes migration safety semantics in production compared with the prior config.

Useful? React with 👍 / 👎.

Comment on lines -1 to -4
# yaml-language-server: $schema=../.moose/migration_schema.json

created_at: 2026-04-08T00:03:51.911046Z
operations: []
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Retain committed migration plan/state snapshots

Removing migrations/plan.yaml (with the matching state snapshots deleted in this commit) breaks the planned-migration workflow used by moose migrate, which applies operations from plan.yaml and performs drift checks against saved state files. Deleting these tracked artifacts removes deterministic migration input and drift protection, so CI/CD migration steps can fail or lose the reviewed-plan guarantees.

Useful? React with 👍 / 👎.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant