Restore Dynamo manifest - #25244
Restore Dynamo manifest#25244piochelepiotr wants to merge 3 commits into
Conversation
6e27900 to
9c6cef3
Compare
evalya-impact-summaryevalya impact analysis |
Disk usage changeCommit No integration or dependency changed size. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0bcb737eb7
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| { | ||
| "manifest_version": "2.0.0", |
There was a problem hiding this comment.
Keep the new Dynamo integration manifestless
Dynamo was introduced in commit 03203e64 without a manifest, so it does not qualify for the exemption for existing integrations that already have one. Adding this file and removing Dynamo's display-name, metrics-prefix, and platform overrides puts the newly created integration back onto the prohibited legacy manifest workflow; retain the three overrides and omit the manifest instead.
AGENTS.md reference: AGENTS.md:L105-L113
Useful? React with 👍 / 👎.
Validation ReportAll 21 validations passed. Show details
|
What does this PR do?
Restores the Dynamo integration manifest, dataflow metadata, and image directory placeholder that were removed for the publishing-platform workflow. Removes the temporary manifestless overrides from
.ddev/config.toml.Motivation
The publishing platform no longer plans to manage internally built integrations, so Dynamo needs to return to the repository-managed manifest workflow.
Review checklist (to be filled by reviewers)
qa/requiredif this PR needs QA validation, orqa/skip-qaif it does not. Exactly one of the two is required.backport/<branch-name>label to the PR and it will automatically open a backport PR once this one is merged