bringyourownideas / silverstripe-composer-security-checker

Provides information if your SilverStripe application uses dependencies with known vulnerabilities.
https://bringyourownideas.com
BSD 3-Clause "New" or "Revised" License
9 stars 6 forks source link

API refactor module to be more independent #34

Closed NightJar closed 6 years ago

NightJar commented 6 years ago

As part of a greater piece of work with site summaries (cf. https://github.com/bringyourownideas/silverstripe-maintenance/issues/39). This commit tidies class definitions and makes the import slightly more efficient. Abstractions are added to relate to the Package object from the aforementioned module. Unit tests have been added to ensure tidy standards in future maintenance.

resolves #1 (although technically it was already resolved).

NightJar commented 6 years ago

Design review is done, this is ready for final review :)

robbieaverill commented 6 years ago

Can you add a branch alias (1.x-dev) to composer.json while you're at it?