danger / danger-js

⚠️ Stop saying "you forgot to …" in code review
http://danger.systems/js/
MIT License
5.24k stars 367 forks source link

Refactor: Put long node error inside html details #1392

Closed glensc closed 1 year ago

glensc commented 1 year ago

when node process errors, that huge dump of details is placed as a note. put it into HTML <details> so the code block is collapsed by default.

image
glensc commented 1 year ago

CI Fix:

orta commented 1 year ago

Cool, yeah, this seems reasonable to me 👍🏻