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.

Sign up for GitHub

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

Jump to bottom

config list still prints env variables #4395

Closed
wdbaruni opened this issue Sep 10, 2024 · 1 comment · Fixed by #4399
Closed

config list still prints env variables #4395

wdbaruni opened this issue Sep 10, 2024 · 1 comment · Fixed by #4399
Assignees
@frrist
Labels
th/game-day Issues reported during game day testing

Comments

@wdbaruni
Copy link
Member 

→ bacalhau-1.5.0 config list
 KEY                                               ENVIRONMENT VARIABLE                                       DESCRIPTION
 api.auth.accesspolicypath                         BACALHAU_API_AUTH_ACCESSPOLICYPATH                         AccessPolicyPath is the path to a file or directory that will be loaded as the p...
 ...
@wdbaruni wdbaruni added the th/game-day Issues reported during game day testing label Sep 10, 2024
@frrist
Copy link
Member

frrist commented Sep 10, 2024

Acceptance Criteria here is to not print the env vars.

frrist pushed a commit that referenced this issue Sep 10, 2024
fix: remove env vars from config list output
0b0a84e 
- provide complete help text for the command.
- close #4395
@frrist frrist mentioned this issue Sep 10, 2024
Merged
@frrist frrist moved this from Inbox to In Review in Engineering Planning Sep 10, 2024
@frrist frrist closed this as completed in 183ee9e Sep 11, 2024
@github-project-automation github-project-automation bot moved this from In Review to Done in Engineering Planning Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@frrist frrist

Labels
th/game-day Issues reported during game day testing
Projects
Engineering Planning
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: remove env vars from config list output bacalhau-project/bacalhau
2 participants
@frrist @wdbaruni