Codecov Report

Merging #111 into master will decrease coverage by 0.03%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #111      +/-   ##
==========================================
- Coverage   80.77%   80.73%   -0.04%     
==========================================
  Files          32       32              
  Lines        5024     5005      -19     
  Branches     1088     1080       -8     
==========================================
- Hits         4058     4041      -17     
  Misses        831      831              
+ Partials      135      133       -2

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f0a7f84...9c86b33. Read the comment docs.

I don't understand what does this fix actually do.

Some overrides, for instance vivado_opts expecting a list of strings, not a string.
ecf673d does not check for that and introduced the regression.

Thanks. I don't think there is any need to dedent overrides provided as a list. Can you change it so that it only dedents strings?

Thanks and sorry for the mess.