tino/pyFirmata — pull request #1 by appultaart (2012-09-18 15:58:25+00:00)
Title: Suggested Part 1: adapt Tino's pyFirmata to python3k code

Hoi Tino,

ik heb mijn commits bekeken, en denk dat de beste strategie om de code review (en eventueel de merge) te doen is via de 'cherry picking' strategie. Ik heb vanaf 'master' (i.e., jouw pyFirmata code met de MIT-licentie update) mijn eerste 5 of 6 commits uit mijn 'py3K' branch in een nieuwe branch 'reviewWithTino' gestopt.

Deze commits zijn niet (nauwelijks) interne functionele aanpassingen, en passen pyFirmata aan aan Py3K, afgaand op de commit logs. Deze eerste pull request geeft mij de kans om te wennen aan hoe 't werkt met Github's pull requests...

(Je zal ondertussen door hebben dat ik geen getraind ITer ben... ;-))

Als deze eerste chunk afgerond is, stel ik voor uit de 'py3k' branch meer van mijn commits toe te voegen: dit zal met name de 'autodetect' en 'capability query' handling worden inclusief de interne code-aanpassingen. Maar dat is voor later.

Als ik het goed begrijp komt er bij jou ook een 'reviewWithTino' branch. De TestSuite kan dan als laatste geschreven worden en ook in die branch komen -- na deze pull request al de TestSuite veranden is wellicht niet zo handig, gezien de aanpassingen in de interne structuur die in deel #2 zullen komen?

Laat evt. andere/betere aanpak horen, ik probeer flexibel te zijn en dit zo makkelijk mogelijk te maken voor je...

Vriendelijke groet,
Douwe