[GitHub] [ant] janmaterne commented on issue #83: Add HTML validation check

2019-03-09 Thread GitBox
janmaterne commented on issue #83: Add HTML validation check URL: https://github.com/apache/ant/pull/83#issuecomment-471177878 Added the redirection to file and did a few renamings. Merged into master. This is an automated mes

[GitHub] [ant] janmaterne commented on issue #83: Add HTML validation check

2019-03-08 Thread GitBox
janmaterne commented on issue #83: Add HTML validation check URL: https://github.com/apache/ant/pull/83#issuecomment-470939094 out|err-props could also work. Catch into prop, write prop to file and print. Give it a try. This

[GitHub] [ant] janmaterne commented on issue #83: Add HTML validation check

2019-03-07 Thread GitBox
janmaterne commented on issue #83: Add HTML validation check URL: https://github.com/apache/ant/pull/83#issuecomment-470836739 This PR can't be responsible for the failing tests as the sources or test files are not touched. Failing tests are: src.tests.antunit.types.resources.

[GitHub] [ant] janmaterne commented on issue #83: Add HTML validation check

2019-03-07 Thread GitBox
janmaterne commented on issue #83: Add HTML validation check URL: https://github.com/apache/ant/pull/83#issuecomment-470832663 Tool "Copyright (c) 2013-2018 Mozilla Foundation" (https://github.com/validator/validator/blob/master/src/nu/validator/client/SimpleCommandLineValidator.java), so

[GitHub] [ant] janmaterne commented on issue #83: Add HTML validation check

2019-03-07 Thread GitBox
janmaterne commented on issue #83: Add HTML validation check URL: https://github.com/apache/ant/pull/83#issuecomment-470825471 Just summarize/quote the link contents: - **doctype**: that declaration was used in HTML5 to distinguish between a standards-compliant parsing mode and a so-call

[GitHub] [ant] janmaterne commented on issue #83: Add HTML validation check

2019-03-07 Thread GitBox
janmaterne commented on issue #83: Add HTML validation check URL: https://github.com/apache/ant/pull/83#issuecomment-470824474 Thanks for the links, sounds reasonable. Two things I found: 1. maybe you should add a short paragraph in the section like for the other tools

[GitHub] [ant] janmaterne commented on issue #83: Add HTML validation check

2019-03-06 Thread GitBox
janmaterne commented on issue #83: Add HTML validation check URL: https://github.com/apache/ant/pull/83#issuecomment-470423575 I like the idea of checking the html files. But why are these changes required? Only because a tool sais that is not enough for me. Do you have pointers to some

[GitHub] [ant] janmaterne commented on issue #83: Add HTML validation check

2019-03-06 Thread GitBox
janmaterne commented on issue #83: Add HTML validation check URL: https://github.com/apache/ant/pull/83#issuecomment-470422494 Summerize the changes: - Add new tool "Nu HTML Validator" for checking manual and site check.xml: new target 'vnu' - add attribute: --> - remov