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

Prints on success #13

Open
alexeagle opened this issue Jun 13, 2022 · 2 comments
Open

Prints on success #13

alexeagle opened this issue Jun 13, 2022 · 2 comments
Labels
blocked Blocked by another issue enhancement New feature or request

Comments

@alexeagle
Copy link
Member

A successful build should be quiet, but I get


INFO: Found 15 targets and 2 test targets...
INFO: From Rollup rollup/tests/index.js:

rollup/tests/gen.js → rollup/tests/index.js...
created rollup/tests/index.js in 5ms

@cgrindel cgrindel added enhancement New feature or request prioritized labels Sep 27, 2022
@gregmagolan gregmagolan moved this to 🔖 Ready in Open Source Feb 6, 2023
@alexeagle alexeagle added this to the 1.0 milestone Mar 27, 2023
@jbedard
Copy link
Member

jbedard commented Apr 20, 2023

There seems to be no easy way to do this today when using the rollup cli without suppressing everything (including warnings etc) using --silent. We have rollup(silent=True|False) to optionally set that flag already. We'd have to write our own script and invoke the rollup API ourselves, essentially https://github.com/rollup/rollup/blob/v3.20.6/cli/run/build.ts (note the logging at the end).

We could disable colors easily though: https://github.com/rollup/rollup/blob/v3.20.6/src/utils/colors.ts#L7

RFC for better logging: rollup/rollup#2393
More logging vs warning/errors discussions: rollup/rollup#1955

@jbedard jbedard added the blocked Blocked by another issue label Apr 21, 2023
@jbedard jbedard removed this from the 1.0 milestone Apr 21, 2023
@sam-mfb
Copy link

sam-mfb commented Aug 3, 2023

If helpful, I posted the workaround we use on the rollup issue 1955

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Blocked by another issue enhancement New feature or request
Projects
Status: On Deck
Development

No branches or pull requests

5 participants