Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Brijeshthummar02
Copy link
Contributor

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.xml that 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.html with working links to the XSD files.

@Brijeshthummar02
Copy link
Contributor Author

GitHub, generate website

@Brijeshthummar02 Brijeshthummar02 changed the title Issue #17865: new xdoc xml page to share with users how to download xsd. Issue #17865: new xdoc xml page to share with users how to download xsd Nov 5, 2025
@Brijeshthummar02
Copy link
Contributor Author

GitHub, generate website

@romani
Copy link
Member

romani commented Nov 5, 2025

#17867 (comment)

I think we need to add new page under https://checkstyle.org/running.html
Call it "Result report". It will have only link xsd and example of xml report for now.

@Brijeshthummar02
Copy link
Contributor Author

GitHub, generate website

@Brijeshthummar02
Copy link
Contributor Author

#17867 (comment)

I think we need to add new page under https://checkstyle.org/running.html
Call it "Result report". It will have only link xsd and example of xml report for now.

Sure, i have added it. let's see how it looks and what further changes we can make in it.

Copy link
Member

@romani romani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Item

@@ -0,0 +1,55 @@
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Member

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

</p>
</section>

<section name="Schema">
Copy link
Member

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.


<section name="Overview">
<p>
Checkstyle can generate XML reports of its findings. These reports
Copy link
Member

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

@Brijeshthummar02
Copy link
Contributor Author

GitHub, generate website

@romani
Copy link
Member

romani commented Nov 6, 2025

Yes, good.

Please fix CI
And do you know why we got this weird indentation
Screenshot_20251105-190940
Is there way to avoid this? We do not have it in other cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants