-
-
Notifications
You must be signed in to change notification settings - Fork 38
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
documentationUrl not used in reports #416
Comments
Ah, this looks to be hardcoded at the moment: vacuum/html-report/ui/src/components/violation-drawer/violation-drawer-component.ts Line 65 in 931a1e1
Should |
This is the first use-case I have heard where someone would like to change the URL in the report. Happy to add this in, could you tell me more about how you would like to use the URL and what you would expect in the html-report and other commands? |
Absolutely! This is a gap I noticed in Spectral as well... they actually reference a common use case for A couple ideas come to mind with regard to html-report:
|
Interesting, sounds like a nice upgrade. |
Despite setting
documentationUrl
in the ruleset, the generated html report still references quobix.com:Sample ruleset:
Sample spec:
Is there another parameter that should be set? Is this the only area using
documentationUrl
for linting errors, or should we see this referenced in CLI output as well?The text was updated successfully, but these errors were encountered: