Hi Raphaël,
welcome aboard! Hope you enjoy contributing to Inkscape. Thanks for your E-mail and contribution.
On Sat, 12 Mar 2016 14:50:15 +0100 Raphaël Bournhonesque <raphael0202@...48...> wrote:
Hello, I would like to contribute to Inkscape, and particularly to the scripting interface. I would like to submit my first patch:
- various typos corrections
- simplify awkward if statement
- move Script::file_listener method definitions in source file
Your patch seems like it does several welcome changes, but one problem I see is that it is not an atomic and self-contained change. In general every commit/changeset should do one thing and do it well, and with being as unintrusive as possible (and without breaking the automated tests or introducing new bugs). Your patch does too many things.
Perhaps consider submitting individual and atomic changes as patches to the launchpad bugtracker.
Regards,
Shlomi Fish