3 ms·
We are insistent because we have first hand knowledge of how Google’s code review and readability processes work.
by enneff 6y ago
We are insistent because we have first hand knowledge of how Google’s code review and readability processes work.
- acqq 6y agoI didn't get that impression, considering the uniqueness of Thompson's engagement: the major point from the start was that he was not an intern who came to add a few lines in existing C++ code. He was there with the bunch of old C (not C++) code which was already written according to different rules, and was to develop a new language for which he was to write the rules (with other such developers). Additionally, he was involved not only in developing that C code, but in initial development of very C language! Under all these circumstances, he apparently couldn't check in the code, for not passing some "test." Your failure to even recognize all that (instead mentioning "C++" and "certification" which is not the same) and to address the fact that these circumstances completely differ from your own much more narrow experience with established practices when checking in C++ code for which the bureaucratic rules already existed simply doesn't tell me that you can be believed regarding organization's capability to not react bureaucratically. Which is what Thompson apparently said and I still find, as I explain, very believable. That is how I searched in the responses for any claim which would directly support your opinion, and still haven't found it. I have no problem to admit I was wrong, but I'd really like to see some direct proof. Like somebody investigating what actually happened there and getting some first hand account. Not the talk about "C++" and existing rules for that (for which there is up to now a confirmation of strong enforcement, not tolerance).
- enneff 6y agoKen is a friend and colleague. I worked with for years on the Go team, from the early days. But ken was at Google a long time before the Go project started. (The Go project was not developed in Google's monorepo and was not subject to Google's readabiliy process, as is the case for many Google projects.) What ken was referring to was indeed the C++ Readability process (which doesn't stop anyone committing code, just requires a review from a person with the 'readability' certification before committing; as mentioned upthread). I know this from talking to him about it, specifically. I can't offer you any more proof other than the fact that I was a witness. If you don't believe that, then I'm sorry? Not sure what else to tell you. FWIW your conduct in this thread is extremely obnoxious. I'm sure you'll find a way to debate that, too, but please consider that I have no vested interest in telling you this other than hoping you might better learn how your conduct is perceived by others.
- acqq 6y ago> I know this from talking to him about it, specifically. Perfect! So what's the exact story, then, finally? 1) Was the quote from him completely invented? 2) Was he allowed to check in or not? Did he need the test to check in at that moment to which the quote refers? (I don't care about the moment when it was published). 3) And even more specifically: were there ever the times where he was in Google but when he not able to check in the code lacking some Google-prescribed "test"? Just "yes", "no" or "I don't know" there, please. Also (just for my curiosity): 4) How long that period lasted? Then you can elaborate where I was wrong by searching thorough what I've written -- I've specifically quoted the article and talked about his C background and involvement in a new language, never claimed anything about his C++ code and a "certification." I have an impression that your perception of what I've written and that, what I actually have written don't match.