|
maze3 wrote: commnet
An NSA term?
"the debugger doesn't tell me anything because this code compiles just fine" - random QA comment
"Facebook is where you tell lies to your friends. Twitter is where you tell the truth to strangers." - chriselst
"I don't drink any more... then again, I don't drink any less." - Mike Mullikins uncle
|
|
|
|
|
Spiders are much more intelligent than smart-phones.
«Beauty is in the eye of the beholder, and it may be necessary from time to time to give a stupid or misinformed beholder a black eye.» Miss Piggy
|
|
|
|
|
For the arachnophobia, or the person telling that?
"If you don't fail at least 90 percent of the time, you're not aiming high enough."
Alan Kay.
|
|
|
|
|
"The horn of Adam" is an example of this type of game?
(5, 6, 7)
modified 26-Jun-17 5:01am.
|
|
|
|
|
First Person Shooter
Adam = first person
and his horn is his hooter - First Person's Hooter.
Andy B
|
|
|
|
|
|
|
![Rose | [Rose]](https://codeproject.freetls.fastly.net/script/Forums/Images/rose.gif)
#SupportHeForShe
Government can give you nothing but what it takes from somebody else. A government big enough to give you everything you want is big enough to take everything you've got, including your freedom.-Ezra Taft Benson
You must accept 1 of 2 basic premises: Either we are alone in the universe or we are not alone. Either way, the implications are staggering!-Wernher von Braun
|
|
|
|
|
At my new work they like each new commit to go through a code review in a shelveset before making it to the dev branch.
Unfortunately code review are done very slowly, sometimes taking up to 2 weeks, and that cause me a lot of trouble when merging back since incompatible change have been introduce since then...
How do you tackle that problem?
One idea which just strikes me is that maybe I could delay my own reviews of my team mate's code, until my own code has been reviewed... this way I prevent introduction of incompatible change...
|
|
|
|
|
Is it supposed to take up to 2 weeks? In general if you're forced to make yourself and by proxy the project less efficient to conform to the standards, the standards are wrong and need to be addressed. Best of luck though!
EDIT: Less efficient within reason, obviously. But if the standard operating procedure leads to constant merge conflicts and a two week delay, that's definitely not within reason in my opinion.
|
|
|
|
|
Code reviews should not take 2 weeks. Your "new work" is doing it wrong.
Super Lloyd wrote: How do you tackle that problem?
If they are doing code reviews in 2 weeks, then what other things are they doing wrong? Talk to the right person about this, I guess, and let them know what you are thinking.
It sounds to me as if your branching methodology may not be correct either, but I have very little to go on here. Remember, if someone else got there code review done before you and there work is QA tested clean and accepted, then you could merge there changes into yours, so that your code "works".
Also, why can't you fix the merge conflicts? It is not a good idea to have multiple people working on separate tasks that revolve around the same code base or bases. That never ends well and is the product of poor project management.
|
|
|
|
|
Taking 2 weeks is just bad. Either your review process way too slow/under-prioritized. Or your commits are way to big. Smaller commits are easier to review. If you have a rule saying "Only 100% functional units can be committed to master" then that rule is counter-productive.
... such stuff as dreams are made on
|
|
|
|
|
megaadam wrote: Smaller commits are easier to review.
|
|
|
|
|
You should re-balance your workload for such intervalls. Do someprototyping classes and functions for the next move.
Maybe I would think about this company is the right for me and my future. 
Press F1 for help or google it.
Greetings from Germany
|
|
|
|
|
Super Lloyd wrote: code review are done very slowly, sometimes taking up to 2 weeks
Two things - 1) that's just stupid, 2) how stupid are the people you work with to tolerate this?
Sorry about my bluntness. 
Marc
Latest Article - Create a Dockerized Python Fiddle Web App
Learning to code with python is like learning to swim with those little arm floaties. It gives you undeserved confidence and will eventually drown you. - DangerBunny
Artificial intelligence is the only remedy for natural stupidity. - CDP1802
|
|
|
|
|
Super Lloyd wrote: sometimes taking up to 2 weeks
Super Lloyd wrote: How do you tackle that problem?
Change the process or get a new job. You can't possibly keep dealing with conflicts like that.
There are two kinds of people in the world: those who can extrapolate from incomplete data.
There are only 10 types of people in the world, those who understand binary and those who don't.
|
|
|
|
|
How often have these code reviews revealed anything worthwhile?
I'm asking as someone who's never gone through a code review.
|
|
|
|
|
To make a blanket statement that 2 weeks to do a code review is wrong is, well... wrong.
What if it is an a nuclear or other critical environment?
Having said that, the code review time frame should be as fast or faster than the development environment. If the code review is taking longer, then management needs to adjust priorities.
I am working on a project that has multiple event based calculations; the process was falling behind because the calculations could to be processed as fast as they were received. We had to rework the calculations to get them processed in a more efficient manner... you have the same situation; your 'events' are code waiting to be reviewed; your 'review' is your calculation... something needs to be adjusted.
|
|
|
|
|
Yeah, the tech lead told us code review are important and must be done first and quick.
I must insist on it being resolved with the other developer when submitted.
plus there is a developer who is particularly slow at them.... and he is going to another team!
|
|
|
|
|
Tim Carmichael wrote: What if it is an a nuclear or other critical environment?
Wow, I can't imaging a code review taking more than a couple of hours, tops.
How long does a nuclear code review take, if you don't mind me asking?
|
|
|
|
|
Think of the complexity of a simple change and everything it touches. Remember DLL hell? Imagine that at a nuclear site; the change must be reviewed from a code perspective, but it must also be reviewed from the perspective of what it can impact.
What happens if the code is put into production with an error? So... a two week review is not unheard of... there is a reason nuclear sites are not running the latest and greatest versions of operating systems.
Now, from a colleague that worked nuclear...
So , for example when I would write and engineering change (EC) package... You are required to get a peer review - they would formally document their review and you had to document how you disposioned each of their comments. After they were happy with how you addressed their comments then they would sign off on your package. The time it took for someone to review the package would take 1/3 - 2/3 the time it took to write it.
Because your review is your signature of the work just as you did it. It is what I know as a true peer review.
After you get a peer review you then would have to get discipline specific reviews done. For example, you have to get a 10CFR50.69 review, I&C Engineering Review, Operations Review, Maintenance review, sometimes even Chemistry and Radiation depends on what I was putting in and who was impacted by my change to the plant design
You have to disposition the comments for each person and they are captured in the package.
|
|
|
|
|
Wow! I am so glad I don't work on nuclear projects.
|
|
|
|
|
I've worked in medical devices for the past 10 years. The longest code review we ever did was a day, and that included checking out the code and performing a full regression test with the physical devices.
Small checkins are good, large checkins are bad. Merging can be challenging enough with a small checkin, depending on what's being changed. Then add a two week time frame and that's a huge disaster waiting to happen.
Two weeks? Time to bring this up with the manager or whoever is in charge. That's ridiculous. 
"Computer games don't affect kids; I mean if Pac-Man affected us as kids, we'd all be running around in darkened rooms, munching magic pills and listening to repetitive electronic music."
-- Marcus Brigstocke, British Comedian
|
|
|
|
|
2 weeks code review is your impediment, you can ask your boss to remove it. Be agile. 
|
|
|
|
|
The flaw in the current process is that the code should really be reviewed "post integration", not pre integration. What if the review is fine, but due to the two week lag the merging/integration introduces a defect? What good did the review do?
"Works on my branch" will become the new mantra.
Maybe all of the merging needs to happen on a "review" branch.
Items are migrated from "review" to "dev" in the order they are committed on "review"
Or else flip it.
Integrate on "dev", migrate via code review to "review" in commit order and build masters from "review" instead of dev.
If someone really wants the fifth package in the "queue" to make it into a build, the preceding four packages will have to be reviewed first.
Another approach would be to perform weekly reviews post label/milestone/sprint. Depending on the churn in certain files, some people would end up reviewing impacts from multiple packages at one time, but I do not see why that would be a problem. This might be more efficient vs. making developers switch more frequently between writing/reviewing.
Also, per another comment. It sounds like the team is not blocking off enough time to perform reviews. For 3 days of coding, you probably need 1 day of review.
Make Monday review day... no new dev work until the review backlog is cleared out.
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
