andk / pause

Perl authors upload server
http://pause.perl.org/
150 stars 57 forks source link

tests: save plack logs to a file. #520

Closed rspier closed 2 months ago

rspier commented 2 months ago

Save the plack logs from tests to a file, and then upload that file as an artifact. This significantly quiets/shortens the output from the tests.

Implement by creating a new plack log environment config ghaction.

rspier commented 2 months ago

@charsbar WrapAction.pm line 28 is one of the remaining sources of logs during the tests. I played with changing it, but only ended up breaking other tests. (Weird that changing something written to STDERR could break tests....) Any thoughts on the best way to send those errors through the logging framework?

I know @rjbs is thinking about long-term logging stuff, but in the short- term I'm just trying to improve developer experiences.

rspier commented 2 months ago

@charsbar I've merged this because it's definitely a dev usability improvement. I'm still interested in improving WrapAction's output, and to know if this was the best way to do it.