Re: [DISCUSS] Add at least test steps to PR

2015-07-23 Thread Daan Hoogland
+1 for less trivial changes that have no clear instruction on testing I would say you would have to be very interested in the component to not just -1 from the start. We are not writing code for the experts. We are writing code for the whole world to use understand and enhance. point of a review

[DISCUSS] Add at least test steps to PR

2015-07-23 Thread Wilder Rodrigues
Hi all, Although we have been through those PR related discussions a lot, I would like to add just 1 point: * When a PR hasn't been properly tested before submitted or when the Committer hasn’t added any test report, that at least the test steps are present in the PR description. If for every