Skip to content

Refactor make_dsubs.py ~~and fix known bugs~~

cpresser requested to merge cpresser/kicad-footprint-generator:dsub into master

See &10

This MR should fix all those issues.

On top of that, the code was formated with black and refactored. The original code used a lot of copy&paste. Now more loops are used. This is an intermediate MR; I have one more thing I want to take care of: get rid of the male/female terminology. But that will be in another MR.

Up to commit bfb2df55 the output of the generator has not changed. I tested this by running the old and new code and then running git diff --no-index old new.

EDIT the last 3 commits were removed. Not this MR only has the refactor.

Only the last 3 commits actually did change the generator output and need review.

Closes the following issues:

Edited by cpresser

Merge request reports