← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Add access to 3d model properties in swig/python

 

Am 01.09.18 um 10:51 schrieb Andrew Lutsenko:
> My test is trivial (create a model with some settings, add it to footprint,
> try to read it back).

Sure the tests are simple, but given there are mostly a dozen of them in
the end count all the single amount of time for getting in touch with
the functions/tests ... it costs a lot of other peoples development time
again and again which is in my eyes just burning time on the wrong side.

And you have mostly written above the minimal needed docstring already.
How much time have you spend to write this btw. :)

As written, I'd like to see ... this doesn't insist it's a needed thing.
This decision is done if, by the members of the project.

So far I can tell I'm always wasting a lot of time in other projects to
understand how the original programmer wanted to use the various
functions and how they work. Documenting functions isn't nice to have,
it's a needed policy for me to work together efficiently today in my
opinion.

> Fixing other tests is outside of scope of this change.

I haven't written you need to fix or document other tests. To say I'm
not responsible for other parts of the source isn't the way FOSS
working, FOSS is working because mostly all feel responsible for 'their'
project.

-- 
Regards
Carsten Schoenert


Follow ups

References