-
-
Notifications
You must be signed in to change notification settings - Fork 237
feat: add new feature to supress the warnings #1805
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
🦋 Changeset detectedLatest commit: e0c358d The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Changeset has been generated for this PR as part of auto-changeset workflow.Please review the changeset before merging the PR.
If you are a maintainer or the author of the PR, you can change the changeset by clicking here Tip If you don't want auto-changeset to run on this PR, you can add the label |
/u |
|
|
…n the validate command
/u |
…e warning is correct or not
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.
Everything else is fine, but I think it would be nice to have support for reading a file, like ignore file for warnings and errors.
Also how would someone know the ids for the warning, do you have any documentation for that.
Co-authored-by: Souvik De <souvikde.ns@gmail.com>
Co-authored-by: Souvik De <souvikde.ns@gmail.com>
Make sense, we can allow file support as well, but we need to consider only warning statements, because errors must be resolved, and we don't have any documentions for this as well |
docs/usage.md
Outdated
@@ -815,10 +815,15 @@ FLAGS | |||
--proxyPort=<value> Port number number for the proxyHost. | |||
--score Compute the score of the AsyncAPI document. Scoring is based on whether the | |||
document has description, license, server and/or channels. | |||
--x-suppress-warnings=<value> One or more warning codes to suppress from diagnostics output. Can be passed |
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.
What's the reasoning for that x-
here? 🤔 Wouldn't it be better to just make it suppress-warnings
or no-warnings
?
That said, I'd even go further and add no-linting
so it removes the warnings and the recommendations but leaves the errors.
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.
yes, need to remove x- here also thanks for idea regarding the supress all warning, we also dicussed in the recent metting as well with @Shurtu-gal i am current implementing it thanks
|
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 🚀
/rtm |
@asyncapi/bounty_team |
Resolves: #1564