Skip to content
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

Infrastructure: HTML validator paths are outdated #2637

Closed
howard-e opened this issue Mar 1, 2023 · 2 comments · Fixed by #2617
Closed

Infrastructure: HTML validator paths are outdated #2637

howard-e opened this issue Mar 1, 2023 · 2 comments · Fixed by #2617
Labels
bug Code defects; not for inaccurate prose Infrastructure Related to maintaining task force and repo operations, processes, systems, documentation

Comments

@howard-e
Copy link
Contributor

howard-e commented Mar 1, 2023

Following the restructure of the repository, the current HTML validator being used is pointing to now absent paths. From package.json:

"vnu-jar": "java -jar node_modules/vnu-jar/build/dist/vnu.jar --filterfile .vnurc --no-langdetect --skip-non-html aria-practices.html index.html examples/"

aria-practices.html index.html examples/ should be content/. This change has been addressed in #2617. But new warnings have been introduced: #2617 (comment)

@howard-e howard-e added bug Code defects; not for inaccurate prose Infrastructure Related to maintaining task force and repo operations, processes, systems, documentation labels Mar 1, 2023
@howard-e
Copy link
Contributor Author

howard-e commented Mar 1, 2023

Looking again at the log, it seems the only new (and non-zero blocking) error is:

example-usage-warning.html":8.9-8.11: error: Element “p” not allowed as child of element “summary” in this context.

Should this be updated or suppressed? cc @alflennik @mcking65

@alflennik
Copy link
Contributor

@howard-e I looked into this an left a comment in the PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Code defects; not for inaccurate prose Infrastructure Related to maintaining task force and repo operations, processes, systems, documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants