Currently, checks are classes that are registered with hook_security_review_checks() and have special methods for returning meta-data about the checks.
Instead, let's:
CheckInterface which Check will implement (should we renamed to CheckBase to match similar systems in D8 core?)ConfigurablePluginInterface rather than accessing config objects directlyThis will make the checks be more inline with similarly extensible systems in D8 core!
As a side benefit, this will make the meta-data about the plugins translatable, because (per best practices) we'll use @Translation() in the annotations.
This replaces previous issues on GitHub: https://github.com/banviktor/security_review/issues/21 and https://github.com/banviktor/security_review/issues/25
Fixed
3.0
Code
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
As this appears to be almost an abandoned ticket. In the 3.0.x branch I got plugins working. Thinking any future enhancements can be handled in follow ups. Saving credit through out the years.
Automatically closed - issue fixed for 2 weeks with no activity.