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.

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

Fix #233: Logging Setup in CLI #284

Closed
wants to merge 1 commit into from
Closed

Fix #233: Logging Setup in CLI #284

wants to merge 1 commit into from

Conversation

uzaxirr
Copy link
Member

@uzaxirr uzaxirr commented Nov 1, 2022

Fixes: #233 Added some logs in root.go and apiKey ls cmd below is how the output will look for the same.


For log in root.go which occurs when a valid sub-cmd is not given
Find the full output here: https://app.warp.dev/block/OUTxstQyNSyW1h1eM5WMup
Screenshot 2022-11-01 at 11 44 17 PM


For the sample log added in apiKey ls cmd
Find the full output over here: https://app.warp.dev/block/lmV9LHSHCV0UpS6BweLKMJ
Screenshot 2022-11-01 at 11 45 45 PM

@uzaxirr
Copy link
Member Author

uzaxirr commented Nov 1, 2022

@RealHarshThakur @alejandrojnm
Please take a look at the output above, if it seems favourable then i shall go ahead and add more relevant logging statements

@alejandrojnm
Copy link
Member

Mm, why is show the help tow times ???

@uzaxirr
Copy link
Member Author

uzaxirr commented Nov 1, 2022

Mm, why is show the help tow times ???

Is it bcz i'm trying to get an error by running go run main.go apikey foo

worked normally when i used go run main.go

Screenshot 2022-11-02 at 12 58 15 AM

Also can you folks suggest me the best way to run and test the CLI locally. I'm currently doing that through the cmds i've shown above

@uzaxirr
Copy link
Member Author

uzaxirr commented Nov 3, 2022

Hey, @RealHarshThakur @alejandrojnm any updates?

@alejandrojnm
Copy link
Member

Hi @uzaxirr I will check this tomorrow morning

@uzaxirr
Copy link
Member Author

uzaxirr commented Nov 7, 2022

Hey @alejandrojnm Any progress?

@haardikdharma10
Copy link
Contributor

Closing due to inactivity. Please reopen if needed.

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.

Logging Setup
3 participants