Skip to content

feat(cli): add --env and --env-file support to improve test coverage #384

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Savvythelegend
Copy link

Title

feat(cli): add --env and --env-file support to improve test coverage and .gitignore

Description

  • Adds support for passing environment variables via --env and --env-file flags to the CLI, enabling easier local testing of cloud functions that depend on runtime env vars.
  • Updates test assertions in test_cli.py for error message accuracy.
  • Updates .gitignore, tox.ini, and .coveragerc for platform-specific coverage support, especially for Windows.
  • No runtime code changes outside CLI/test/config improvements.
  • Addresses Ability to pass runtime environment variables for local testing #215.

…ve test coverage and .gitignore\n\n- Adds support for passing environment variables via --env and --env-file flags to the CLI, enabling easier local testing of cloud functions that depend on runtime env vars.\n- Updates test assertions in test_cli.py for error message accuracy.\n- Updates tox.ini and .coveragerc for platform-specific coverage, especially for Windows.\n- No runtime code changes outside CLI/test/config improvements.\n- Addresses GoogleCloudPlatform#215.
@Savvythelegend Savvythelegend force-pushed the feat/env-vars-from-cli branch from 8af8826 to 9e1db58 Compare July 6, 2025 10:37
@Savvythelegend
Copy link
Author

Savvythelegend commented Jul 9, 2025

Hi @josephlewis42 @dazuma @jskeet @chingor13 @garye, just following up on this PR — it adds --env and --env-file support to the CLI for easier local testing of env-dependent functions (#215).
Would appreciate any feedback or a review when possible.

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.

1 participant