svenkreiss / html5validator

Command line tool to validate HTML5 files. Great for continuous integration.
MIT License
314 stars 34 forks source link

Added log file creation option. #66

Closed Cyb3r-Jak3 closed 4 years ago

Cyb3r-Jak3 commented 4 years ago

Added argument --log-file that if set changes the output of the log to a file rather than stout. Addtionally, linked option for GitHub action that I created,

svenkreiss commented 4 years ago

Thanks for sending the PR. What is the difference to using tee? Like html5validator someoptions | tee test.log?

Cyb3r-Jak3 commented 4 years ago

In my tests, neither tee or > worked to generate a log file.

NFNncn2XsX

Cyb3r-Jak3 commented 4 years ago

Upon further investigation, using html5validator option &> log.log will both show in the terminal and create a log file.

Up to you if you would rather have log file creation in or out of scope.

svenkreiss commented 4 years ago

Thanks for following up. The &> seems to be a good solution.