Hi, any code contributor is expecting that their patches will be tested by the reviewer? If that is the case, we should make more clear in the review process that patches should have been tested _before_ they are submitted for acceptance.
Because of this specific commit, file transfers have been broken since early this year and it's obvious that this code wasn't tested at all: http://git.sugarlabs.org/projects/sugar/repos/mainline/commits/11828796 Given the current poor state of our testing efforts, committers should be extra careful about this. Regards, Tomeu _______________________________________________ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel
