Add support for list overrides

LinkedSoftwareDependencies / Components.js

🧩 A semantic dependency injection framework

https://componentsjs.readthedocs.io/

Other

41 stars 6 forks source link

Add support for list overrides #134

Closed joachimvh closed 8 months ago

joachimvh commented 9 months ago

Closes https://github.com/LinkedSoftwareDependencies/Components.js/issues/129

Implements what is described in https://github.com/LinkedSoftwareDependencies/Components.js/issues/129#issuecomment-1814630077

Will look into documentation when this is accepted.

Some of the behaviour that happened in the canHandle of the ConfigPreprocessorOverride was moved to the transform call, but this should not have an impact.

coveralls commented 9 months ago

Pull Request Test Coverage Report for Build 7061626539

162 of 162 (100.0%) changed or added relevant lines in 9 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 7050284425:	0.0%
Covered Lines:	1493
Relevant Lines:	1493

💛 - Coveralls

joachimvh commented 9 months ago

Everything seems to work fine when testing with Comunica.

rubensworks commented 8 months ago

Released as 5.5.0.