Skip to content
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.

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

Add GitHub action env variable deprecation warning #550

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

jasonforal
Copy link
Collaborator

What problem are you trying to solve?

In #545, we removed support for the DD_SERVICE and DD_ENV environment variables in the GitHub action.

Users may still be passing these environment variables in through the use of our official GitHub action as a part of their workflows, but they will have no effect.

What is your solution?

Warn the user that the environment variable has no effect using GitHub's special "warning message".

Alternatives considered

What the reviewer should know

@jasonforal jasonforal requested a review from a team as a code owner November 5, 2024 09:21
@jasonforal jasonforal requested review from jacobotb and amaanq and removed request for jacobotb November 5, 2024 09:21
@jasonforal jasonforal merged commit b5bdd4b into main Nov 6, 2024
70 checks passed
@jasonforal jasonforal deleted the jf/deprecation branch November 6, 2024 16:57
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