Skip to content

fix: remove DRY_RUN env var and --dry-run flag - #210

Merged
anniefu merged 1 commit into
GoogleCloudPlatform:masterfrom
anniefu:noDryRun
Dec 7, 2022
Merged

fix: remove DRY_RUN env var and --dry-run flag#210
anniefu merged 1 commit into
GoogleCloudPlatform:masterfrom
anniefu:noDryRun

Conversation

@anniefu

Copy link
Copy Markdown
Contributor

Free the DRY_RUN env var name to be used for other purposes by function authors.

The original intent of the DRY_RUN was to be used at build time for GCF to validate function syntax without starting the server, but this was never implemented. For local testing purposes, simply starting the functions framework server is a better method.

Free the DRY_RUN env var name to be used for other purposes by function
authors.
The original intent of the DRY_RUN was to be used at build time for GCF
to validate function syntax without starting the server, but this was
never implemented. For local testing purposes, simply starting the
functions framework server is a better method.
@anniefu
anniefu requested a review from a teamDecember 6, 2022 22:04
@anniefu
anniefu merged commit f013ab4 into GoogleCloudPlatform:masterDec 7, 2022
@release-pleaserelease-pleaseBot mentioned this pull request Dec 7, 2022
Sign up for freeto 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

@anniefu@jinglundong