search

hildogjr / KiCost

Build cost spreadsheet for a KiCad project.
MIT License
519 stars 98 forks source link

Fix qty computation #456

Closed set-soft closed 3 years ago

set-soft commented 3 years ago

This patch fixes the compuation problem explained in #455

set-soft commented 3 years ago

Hi @hildogjr !

Any comment about this patch?

I added a new test to check the variants, kicost.VARIANT:FIELD, DNP and version -> variant / nopop -> dnp rename. But I'm not creating a pull-request because the test needs code used for the multiproject test.

hildogjr commented 3 years ago

Thanks @set-soft. I just see that I also need make a review of the general KiCost code to keep moving forward.