appfolio / gemsurance

Gem vulnerability checker using rubysec/ruby-advisory-db
MIT License
213 stars 24 forks source link

Added YML output format #16

Closed bencolon closed 9 years ago

bencolon commented 9 years ago

Hey Jon, a cleaner PR to support YML output format. Thoughts ?

jonkessler commented 9 years ago

Looks better. Could you add tests for BaseFormatter and YmlFormatter, please? Also, perhaps vulnerabilities would be a better name for the details_status key in the YAML. Thanks.

bencolon commented 9 years ago

Hi Jon. Vulnerabilities key updated and Formatters tests added.

jonkessler commented 9 years ago

Looks good. Thanks for the contribution!