Skip to content

Conversation

@Dexterp37
Copy link
Contributor

Instead of printing the whole exception, this exclusively prints stdout and stderr.

In case of error, here's what gets printed:

npx glean glinter src/*.yaml
Checking for a Glean virtual environment at /mnt/c/Mozilla/glean.js/samples/web-extension/javascript/.venv
Using Glean virtual environment at /mnt/c/Mozilla/glean.js/samples/web-extension/javascript/.venv
ERROR running glean_parser v2.5.0

==============================================================================
/mnt/c/Mozilla/glean.js/samples/web-extension/javascript/src/metrics.yaml:
    ```
    sample: webext_started!
    ```

    'webext_started!' is not valid under any of the given schemas
        'webext_started!' does not match '^[a-z_][a-z0-9_]*$'

Instead of printing the whole exception, this exclusively
prints stdout and stderr.
@Dexterp37 Dexterp37 requested a review from brizental April 16, 2021 16:13
@Dexterp37 Dexterp37 self-assigned this Apr 16, 2021
@Dexterp37 Dexterp37 merged commit 678a833 into mozilla:main Apr 19, 2021
@Dexterp37 Dexterp37 deleted the cli_err branch April 19, 2021 08:10
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.

2 participants