testXHTML: remove W3Validator
W3Validator seems obsolete now since it does not validate HTML5. nu became the default validator in 417ff1ae.
Merged
by 
The changes were merged into master. The source branch has been removed.
-
This looks good. Tests are running as https://nexedi.erp5.net/test_result_module/20181025-1F06BBEF/ ( because I pushed in test runner branch , we don't have test running automatically on merge requests ).
Next, we can also remove it from slapos software release ( here and here ).
-
855 777 tested_portal_type_list=tested_portal_type_list) 856 778 857 779 858 # Three validators are available : nu, tidy and the w3c validator 780 # Two validators are available : nu and tidy validator 859 781 # It's hightly recommanded to use the nu validator which validates html5 860 782 validator_to_use = 'nu' 861 783 show_warnings = True 862 784 863 785 validator = None 864 786 865 # tidy or w3c may not be installed in livecd. Then we will skip xhtml validation tests. 787 # tidy may not be installed in livecd. Then we will skip xhtml validation tests. -
Tests are running as https://nexedi.erp5.net/test_result_module/20181025-1F06BBEF/
there are failures, but they are unrelated, master has the same failures sometimes.
-
Toggle commit list
-
unmarked as a Work In Progress
Toggle commit list -
changed title from Remove W3Validator to testXHTML: remove W3Validator
Toggle commit list -
Any reason this is not merged ?
-
No, I think it can be merged.
-
merged
Toggle commit list
-
Please register or sign in to post a comment