Financial-Times / n-ui / 9655

89%

master: 91%

Build Type

push

circle-ci

Committed by Andrew Georgiou

Commit Message

o-ads v13 and above does not contain the krux module as this was deprecated; however Permutive makes use of some data from the ads-api which is still namespaced under 'krux'; this PR keeps backwards compatibility with v1 ads-api but allows us to update to v14 o-ads. Ideally we will either move n-ui to v2 ads-api where all references to krux have been removed; or roll-out pagekit and deprecate n-ui. (Pagekit is being updated to v2 ads-api seperately) 🐿 v2.12.5

Run Details

132 of 179 branches covered (73.74%)

341 of 385 relevant lines covered (88.57%)

30.04 hits per line