Hi! > This seems like a huge burden to developers writing non-trivial code. > What does "very clear" mean? > > Proper reviewing includes reading and understanding the actual code. If > code is unclear, reviewers ask for clarification. Now, apart from a > concept that has been approved, perfect doxygen, commented code, there > needs to be code meta documentation that makes "very clear" how the code > works. > > This might speed up PR reviewing, but it slows PR creation. Let's not > lose focus, we want to streamline RIOT development, not PR reviewing.
Now it's up to me to ask: are you trying to piss me off? I see this actually as one of the most important rules. If the code is complex and poorly documented, it sucks. Full stop! Do you know how many hours I spent in trying to understand poorly documented, but complex (and not always brilliant) code? And do you know how many times I did this not only once, while reviewing, but hundreds of times while debugging code or reviewing new PRs that changes this code? Sorry, Kaspar, but I won't review any poorly documented code any more. It's waste of time. Cheers, Oleg P.S. Sorry for the harsh words, but this attitude drives me nuts. -- prom_printf("Detected PenguinPages, getting out of here.\n"); linux-2.0.38/arch/sparc/mm/srmmu.c
signature.asc
Description: PGP signature
_______________________________________________ devel mailing list devel@riot-os.org https://lists.riot-os.org/mailman/listinfo/devel