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

Add --no-print-progress flag to gleam run and gleam build #2299

Closed
PastMoments opened this issue Aug 16, 2023 · 9 comments
Closed

Add --no-print-progress flag to gleam run and gleam build #2299

PastMoments opened this issue Aug 16, 2023 · 9 comments
Labels
good first issue Good for newcomers help wanted Contributions encouraged priority:medium

Comments

@PastMoments
Copy link

This is especially useful for piping program's stdout to other things.

Brief discussion about it here:
https://discord.com/channels/768594524158427167/1139046296439758858

@lpil
Copy link
Member

lpil commented Aug 20, 2023

Good suggestion, thank you. I think I would like to use a more verbose flag as --silent is is more likely to be something the Gleam program would want to use. Perhaps --no-print-progress?

This would be implemented by passing in the null telemetry instead of the one that prints to the console.

@lpil lpil added help wanted Contributions encouraged good first issue Good for newcomers priority:medium labels Aug 20, 2023
@inoas
Copy link
Contributor

inoas commented Aug 20, 2023

Would this be only about the stdout infos or also about warnings?

@lpil
Copy link
Member

lpil commented Aug 21, 2023

Only the progress information, warnings and errors would still be printed.

@ohamill
Copy link

ohamill commented Dec 2, 2023

I'd like to work this ticket, if possible. Should I proceed with naming the flag --no-print-progress?

@lpil
Copy link
Member

lpil commented Dec 4, 2023

Yes please 🙏

@erikareads
Copy link
Contributor

I would love this!

@codereport
Copy link

Not sure if the --silent would suppress warnings as well but that would be very helpful.

@lpil
Copy link
Member

lpil commented Aug 1, 2024

It would not, no.

@lpil lpil changed the title Add --silent flag to gleam run and gleam build Add --no-print-progress flag to gleam run and gleam build Aug 1, 2024
@inoas
Copy link
Contributor

inoas commented Aug 2, 2024

--warnings-as-errors would also be helpful for CI purposes (modifying the posix exit code to be 0)
I am personally agains --no-warnings

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Contributions encouraged priority:medium
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants