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

Stop silencing errors during fuels init command #2483

Closed
dmihal opened this issue Jun 7, 2024 · 3 comments · Fixed by #2558
Closed

Stop silencing errors during fuels init command #2483

dmihal opened this issue Jun 7, 2024 · 3 comments · Fixed by #2558
Assignees
Labels
bug Issue is a bug
Milestone

Comments

@dmihal
Copy link
Contributor

dmihal commented Jun 7, 2024

I was starting a new TS workspace and ran pnpm fuels init, which kept failing to create any output.

Eventually I noticed an error was briefly flashing on my terminal before being erased. Using a screen recorder, I could see the following error was flashing:

Error: required option '-w --workspace <path>' not specified

First of all, the docs should provide more information about the required flags.

Second, the CLI should make sure errors like this are visible to the user (without needing a screen recorder 😁).

@arboleya arboleya added the bug Issue is a bug label Jun 9, 2024
@arboleya
Copy link
Member

arboleya commented Jun 9, 2024

It shouldn't require a screen recorder, indeed! 😆

The docs page you mentioned contains the URL for the init command docs though:

nmext-step

Here it is:

Please let me know if you think something feels off.

@arboleya arboleya changed the title CLI: fuels init fails and error not visible Command fuels init fails silently Jun 9, 2024
@arboleya arboleya added this to the Mainnet milestone Jun 9, 2024
@arboleya arboleya removed the mainnet label Jun 9, 2024
@arboleya arboleya changed the title Command fuels init fails silently Stop silencing errors during fuels init command Jun 10, 2024
@Dhaiwat10
Copy link
Member

@arboleya To be fair, that link can look like just another code snippet and people might not click on it. They might just think that pnpm fuels init is gonna be all they need to run.

@arboleya
Copy link
Member

@Dhaiwat10 Agreed. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue is a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants