-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
Issue #17865: new xdoc xml page to share with users how to download xsd #18054
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
base: master
Are you sure you want to change the base?
Conversation
|
GitHub, generate website |
|
GitHub, generate website |
|
|
GitHub, generate website |
Sure, i have added it. let's see how it looks and what further changes we can make in it. |
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.
Item
src/site/xdoc/xsds.xml
Outdated
| @@ -0,0 +1,55 @@ | |||
| <?xml version="1.0" encoding="UTF-8"?> | |||
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.
I don't see value in this page.
Please remove.
Report xsd is not configuration, link to xsd of configuration we already have in section where we describe config structure
src/site/xdoc/result_report.xml
Outdated
| </p> | ||
| </section> | ||
|
|
||
| <section name="Schema"> |
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.
Please name it "Xml report" section and combine following example section into this section.
So in future there will be section for all reports, but not in this PR.
src/site/xdoc/result_report.xml
Outdated
|
|
||
| <section name="Overview"> | ||
| <p> | ||
| Checkstyle can generate XML reports of its findings. These reports |
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.
Checkstyle can generate XML, SARIF and plain text reports of its findings.
Reference to xsd should be moved to xml report section
|
GitHub, generate website |

Part of Issue #17867 , Comment #17865 (comment)
This change adds a new page to the Checkstyle website where users can find and download the XSD files for validating configuration files and report outputs.
What I Did
Made a new file
src/site/xdoc/xsds.xmlthat lists the XSDs with links to download them.Added "XSD Schemas" to the menu under "Configuration" in
src/site/site.xml.Why
Users need a way to get the XSDs easily, just like the DTDs.
The XSDs are already set to be copied to the site by the build process.
This page explains how to use them in XML files.
The new page shows up at
target/site/xsds.htmlwith working links to the XSD files.