Financial-Times / content-unroller / 371

73%

Build Type

push

circleci

Committed by web-flow

Commit Message

Uppsf 5164 extend content unroller to handle unrolling of clip set and clip (#39)

* Updating Pull Request Template.

* Added clipset test

* Searated unrollers in structs

* Split unrollers into separate files

* Simplified unroller interface

* Improved Error handling

* Made handler work with slice of unrollers

* Added clipset unrolled

* Added clip unrolled

* Added imageSet unroller

* Fixed tests

* Repaced unroller structs with functions

* Relcased old logger with UPPLogger

* Added tests

* Addressed todos

* Fixed linters

* Added format field to member response

* experimented with reading clipset before unroll

* Revert "experimented with reading clipset before unroll"

This reverts commit f8927280d.

* Addressed comments

---------

Co-authored-by: ft-upp-team <ft-upp-team@ft.com>

Run Details

391 of 519 new or added lines in 13 files covered. (75.34%)

10 existing lines in 2 files now uncovered.

658 of 905 relevant lines covered (72.71%)

14.44 hits per line