Skip to content

Comments

Allow passing version name via environmental variable to apps plugin#272

Merged
yoannmoinet merged 1 commit intomasterfrom
oliver/allow-passing-version-name-via-environmental-variable-to-apps-plugin
Feb 20, 2026
Merged

Allow passing version name via environmental variable to apps plugin#272
yoannmoinet merged 1 commit intomasterfrom
oliver/allow-passing-version-name-via-environmental-variable-to-apps-plugin

Conversation

@oliverli
Copy link
Collaborator

@oliverli oliverli commented Feb 19, 2026

What and why?

Updates the apps plugin to take in an optional version name via the DATADOG_APPS_VERSION_NAME environmental variable when uploading app assets.

The intent is to use this variable name in CI (setting it to the Git commit SHA) to identify each version that is uploaded.

If unspecified, defaults to server generated version names.

How?

Updated getData to fetch the DATADOG_APPS_VERSION_NAME environmental variable, setting it in the multipart form if present

Copy link
Collaborator Author

oliverli commented Feb 19, 2026

@oliverli oliverli force-pushed the oliver/allow-passing-version-name-via-environmental-variable-to-apps-plugin branch from da2826c to a6d0bae Compare February 19, 2026 21:12
@oliverli
Copy link
Collaborator Author

@cursor review
@codex review

Copy link

@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: a6d0bae0df

ℹ️ 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".

@oliverli oliverli marked this pull request as ready for review February 19, 2026 21:18
Copy link
Member

@yoannmoinet yoannmoinet left a comment

Choose a reason for hiding this comment

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

I don't think we need to define it in validatedOptions, but only check for it at the very end.
Let me know if there actually is a reason for it.

@oliverli oliverli force-pushed the oliver/allow-passing-version-name-via-environmental-variable-to-apps-plugin branch from a6d0bae to 008e24b Compare February 20, 2026 05:53
@oliverli oliverli requested a review from yoannmoinet February 20, 2026 06:00
Copy link
Member

@yoannmoinet yoannmoinet left a comment

Choose a reason for hiding this comment

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

Thanks, looks good 👍

@yoannmoinet yoannmoinet merged commit 8f13baa into master Feb 20, 2026
5 checks passed
@yoannmoinet yoannmoinet deleted the oliver/allow-passing-version-name-via-environmental-variable-to-apps-plugin branch February 20, 2026 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants