-
-
Notifications
You must be signed in to change notification settings - Fork 248
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
Can't silence reports #1018
Comments
@m-mohr Awesome feedback! <3 You're right, the doco may need a little love. Escaping in Json Pointer land is a little bit unwelcoming (cf. https://tools.ietf.org/html/rfc6901#section-3). Applying those (a little bit) unfriendly rules, the ruleset should look like:
Which now gets correctly interpreted. However, your use case led me to discover a bug in the code. Using this ruleset, linting the document from https://raw.githubusercontent.com/Open-EO/openeo-api/spectral/openapi.yaml returns ... no result. Duh! The result coming from "/service_types" is swallowed 😢 I'm working on a patch. Hardest part of it should be coming with a simple repro case that I could fit into a unit test. Stay tuned! @P0lip I hope @stoplight-bot the release bot is unaffected by the current pandemic as we may need some help from it to issue a patch release with the fix 😜 |
@nulltoken Thanks! I just tried to use it with your changes from the PR. With your two except rules I got rid of two errors, but I still get two other errors:
So now I'm trying with the following rules:
Unfortunately, I still get the two errors and the paths are taken from the |
Glad that it helped :-) Thanks for the quick fix. |
Fix #1018 Co-authored-by: Jakub Rożek <[email protected]>
Describe the bug
I just installed 5.2.0 with the new silencing feature introduced by @nulltoken. I'm trying to silence the errors I am getting for my openapi.yaml described here: #915
To start with broad example, I added
Unfortunately, it still shows the errors. I also tried some more paths, but except for an error that the source is missing for path
#
nothing changed.What I really want to achieve:
Although for this example I'm not sure whether the forward slashes are correctly escaped.
I'm using JSON Reference here because the example in the docs look like JSON Reference although the doc says JSON Path. I'm confused, but tried both...
To Reproduce
See #915 and add the code given above.
Expected behavior
Errors should be silent. Either I don't understand the new feature/docs and that needs improvement or there's a bug in the code.
Environment (remove any that are not applicable):
The text was updated successfully, but these errors were encountered: