-
Notifications
You must be signed in to change notification settings - Fork 7
feat(cli): Add 'quiet' option to report errors only #629
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
Conversation
had some trouble with |
Thanks a lot for your contribution @marianfoo. We will take care of this PR in backlog item CPOUI5FOUNDATION-837. |
521fcbc
to
b10be3d
Compare
Also fix minor eslint error.
b10be3d
to
78d3731
Compare
Rebased, snapshots updated (using Let's get a review from UA and from another developer of the team. |
Sorry to bother you @marianfoo, but since we moved the repository to a new organization, you'll need to sign the CLA again (see comment above) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thank you @RandomByte for the final changes and the team for approving and merging my PR! |
UI5 linter reports findings declared as warnings and errors. There are scenarios where only findings declared as an error should be reported. ESLint offers also an CLI option "quiet" to achieve this.
JIRA: CPOUI5FOUNDATION-837