Skip to content

Comments

Default apps dryRun to true#271

Merged
yoannmoinet merged 2 commits intomasterfrom
yoann/apps-dryrun-default-true
Feb 18, 2026
Merged

Default apps dryRun to true#271
yoannmoinet merged 2 commits intomasterfrom
yoann/apps-dryrun-default-true

Conversation

@yoannmoinet
Copy link
Member

@yoannmoinet yoannmoinet commented Feb 17, 2026

What and why?

Change apps.dryRun default from false to true so asset uploads are opt-in rather than happening on every build.

How?

  • validate.ts now defaults dryRun to !getDDEnvValue('APPS_UPLOAD_ASSETS') instead of false
  • Added APPS_UPLOAD_ASSETS to the env override variables in @dd/core
  • Updated tests and README accordingly

Uploads are now opt-in via `dryRun: false` or the
`DATADOG_APPS_UPLOAD_ASSETS` / `DD_APPS_UPLOAD_ASSETS` env var.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@yoannmoinet yoannmoinet marked this pull request as ready for review February 17, 2026 19:38
@yoannmoinet yoannmoinet merged commit 331c83f into master Feb 18, 2026
4 checks passed
@yoannmoinet yoannmoinet deleted the yoann/apps-dryrun-default-true branch February 18, 2026 14:06
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.

2 participants