Skip to content

Migrate validate config from datadog_checks_dev to ddev #23653

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
AAraKKe wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Migrate validate config from datadog_checks_dev to ddev #23653

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
331604c
Migrate validate config from datadog_checks_dev to ddev
AAraKKe May 10, 2026
50130ef
Add changelog
AAraKKe May 10, 2026
ae39341
Apply formatter
AAraKKe May 10, 2026
364a360
Apply type annotations to satisfy mypy
AAraKKe May 10, 2026
5f259a4
Pass 'all' sentinel to iter_all() so no-arg and 'all' branches iterat…
AAraKKe May 10, 2026
55099e4
Use iter_packages('all') to match legacy get_valid_checks semantics
AAraKKe May 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions datadog_checks_dev/changelog.d/23653.changed
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Legacy migration: remove `validate config` from the legacy CLI; the command is now provided natively by ddev.
2 changes: 0 additions & 2 deletions datadog_checks_dev/datadog_checks/dev/tooling/commands/validate/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
from .agent_reqs import agent_reqs
from .agent_signature import legacy_signature
from .codeowners import codeowners
from .config import config
from .dashboards import dashboards
from .dep import dep
from .eula import eula
Expand All @@ -25,7 +24,6 @@
ALL_COMMANDS = (
agent_reqs,
codeowners,
config,
dashboards,
dep,
eula,
Expand Down
247 changes: 0 additions & 247 deletions datadog_checks_dev/datadog_checks/dev/tooling/commands/validate/config.py
View file
Open in desktop

This file was deleted.

73 changes: 0 additions & 73 deletions datadog_checks_dev/tests/tooling/commands/validate/test_config.py
View file
Open in desktop

This file was deleted.

1 change: 1 addition & 0 deletions ddev/changelog.d/23653.added
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Legacy migration: `validate config` is now implemented natively in ddev (was previously delegated to datadog_checks_dev).
2 changes: 1 addition & 1 deletion ddev/src/ddev/cli/validate/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
from datadog_checks.dev.tooling.commands.validate.agent_reqs import agent_reqs
from datadog_checks.dev.tooling.commands.validate.agent_signature import legacy_signature
from datadog_checks.dev.tooling.commands.validate.codeowners import codeowners
from datadog_checks.dev.tooling.commands.validate.config import config
from datadog_checks.dev.tooling.commands.validate.dashboards import dashboards
from datadog_checks.dev.tooling.commands.validate.dep import dep
from datadog_checks.dev.tooling.commands.validate.eula import eula
Expand All @@ -22,6 +21,7 @@

from ddev.cli.validate.all import all
from ddev.cli.validate.ci import ci
from ddev.cli.validate.config import config
from ddev.cli.validate.http import http
from ddev.cli.validate.labeler import labeler
from ddev.cli.validate.licenses import licenses
Expand Down
Loading
Loading