Le mercredi 14 mars 2018 à 10:04:56+0100, Pierre-Elliott Bécue a écrit : > Hi, > > Thanks for submitting a patch, and for your contribution to this bug. > > Your patch works proprely on the debian's yaml file, but it appears that you > didn't run the tests after designing it. > > Some general recommendations: > > * map(function, iterable) is mostly obsolete, use [function(e) for e in > iterable] instead, especially in django applications as python3 version > of map returns an iterator that django is unable to serialize. > * Your test yaml files were having two issues: > + No entry in the 'excuses' entry. > + 'policy-info' should have been 'policy_info' > * You didn't update expected_data properly in the tests (with yaml, > integers are casted as integers, and thus in the test file, '20' and '10' > should have been replaced with 20 and 10) > * set_update_excuses_content is supposed to call yaml.load() after your > changes. It didn't. > > Thanks for your contribution. :) Oh, by the way, I'll take care of the rest and the integration. :) Cheers, -- Pierre-Elliott Bécue GPG: 9AE0 4D98 6400 E3B6 7528 F493 0D44 2664 1949 74E2 It's far easier to fight for one's principles than to live up to them.
Attachment:
signature.asc
Description: PGP signature