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
Use module properties macro in all templates #13693
Comments
We can not migrate modules that has tokens as property as xdoc validation demands them to be the last in table. But for now macros use simple alphabetical ordering |
#13666 should be done first for each page before adding the macro to the page. The re-ordering should be a manual process of moving the property to a new position so we are sure the new macro isn't causing issues, as we found one. |
List of remaining items is at checkstyle/config/checkstyle-non-main-files-suppressions.xml Lines 230 to 604 in d316b89
|
…essages, final fix
macros is implemented at #13422
we need to use it in all templates
this is example of update we need to make for template files - https://github.com/checkstyle/checkstyle/pull/13681/files#diff-01387dd92ef7af5a9a1f274744b44e12fbeda8302eb79072785f7d6c9d559727L40
mvn clean test -Dtest=XdocsPagesTest,XdocsJavaDocsTest
The text was updated successfully, but these errors were encountered: