Skip to content

docs: update DENO_DEPLOY_APPLICATION_SLUG to DENO_DEPLOY_APP_SLUG#3012

Merged
crowlKats merged 3 commits intodenoland:mainfrom
gentlementlegen:patch-1
Mar 31, 2026
Merged

docs: update DENO_DEPLOY_APPLICATION_SLUG to DENO_DEPLOY_APP_SLUG#3012
crowlKats merged 3 commits intodenoland:mainfrom
gentlementlegen:patch-1

Conversation

@gentlementlegen
Copy link
Copy Markdown
Contributor

The documentation states that the variable DENO_DEPLOY_APPLICATION_SLUG contains the human-readable slug for the app, whereas DENO_DEPLOY_APP_SLUG seems to be the variable that is actually populated.

This can be checked creating an App and adding a build step with:

echo "$DENO_DEPLOY_APP_SLUG"

which will yield the proper value, while

echo "$DENO_DEPLOY_APPLICATION_SLUG"

Will yield nothing.

@crowlKats
Copy link
Copy Markdown
Member

crowlKats commented Mar 31, 2026

@gentlementlegen thanks for this; would you be able to update the other env vars too?
they hsould be

DENO_DEPLOY_ORG_ID
DENO_DEPLOY_ORG_SLUG
DENO_DEPLOY_APP_ID

@gentlementlegen
Copy link
Copy Markdown
Contributor Author

@crowlKats Updated in b9d3046

@crowlKats
Copy link
Copy Markdown
Member

@gentlementlegen thanks! could you run the formatter, and then we can land this

@gentlementlegen
Copy link
Copy Markdown
Contributor Author

@crowlKats CI is green now!

Copy link
Copy Markdown
Member

@crowlKats crowlKats left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@crowlKats crowlKats merged commit 189e62d into denoland:main Mar 31, 2026
1 check passed
@gentlementlegen gentlementlegen deleted the patch-1 branch March 31, 2026 11:26
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