Code review comment for lp:~sylvain-pineau/checkbox/json_load_classmethods_bis

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

73 + #if value.job is not self.job:
74 + # raise ValueError("result job does not match")

I probably know why you did this but I don't know if just disabling that (especially this way) is okay. Could you explain this please?

« Back to merge proposal