Inject Properties class into Linter to respect Dependency Inversion principle

Question

Closed this issue 4 years ago · 2 comments

Currently the Properties class is not declared as final but it's impossible to inject an extended class of it into the Linter.

For this I would propose to add a constructor to Linter and remove the getCssLintProperties() method.

Answer 1 · 2019-11-08T10:46:18.000Z

I've added a constructor to Linter, that accept an optional instance of the "\CssLint\Properties" helper (Version 2.1.0)

Answer 2 · 2020-09-18T01:31:49.000Z

Stale issue message