Skip to content

fix: remove print user info#385

Merged
anayeaye merged 2 commits into
mainfrom
fix/rm-print-statement
Jul 11, 2025
Merged

fix: remove print user info#385
anayeaye merged 2 commits into
mainfrom
fix/rm-print-statement

Conversation

@anayeaye

@anayeaye anayeaye commented Jul 1, 2025

Copy link
Copy Markdown
Contributor

What

Do print informative statement if env variables are missing but do not print values

How Tested

>>> stage="production"
>>> username_env=None
>>> password_env="pants"                                                                                                   
>>> print(f"Missing one or more environment variables: stage undefined={stage==None}, username undefined={username_env==None}, password undefined={password_env==None}")

Missing one or more environment variables: stage undefined=False, username undefined=True, password undefined=False

@anayeaye anayeaye requested a review from smohiudd as a code owner July 1, 2025 18:10
@anayeaye anayeaye requested a review from botanical July 1, 2025 18:11
@anayeaye anayeaye merged commit e047e80 into main Jul 11, 2025
2 checks passed
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