Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Is there any reason why you all want to be notify whenever someone commit some codes?

I know there are things like doing manual check to make sure the codes committed is fundamentally correct. But beside that? Reason i asking is because a big project there are so many commits everyday.

like image 450
Nick Long Avatar asked Oct 10 '08 01:10

Nick Long


4 Answers

Generally speaking commit notifications are useful so that you have some idea what's happening in the code on a daily basis. For example, I know when a new feature has been implemented, or a bug fix is checked in. If there's so many commits that this gets lost in the noise then it might be worth filtering or reducing the number of commit messages you see (perhaps you can limit it only to specific modules you work on, for example).

That being said, aside from giving other developers the opportunity to at least cursorily review code, it can also notify people that:

  • it's a good time to run an update from source control and merge any new changes if needed
  • Hey, someone found a regex parsing bug in Foo but forgot to update Bar with it
  • Oh, Bob's working on the Baz module, I should ask him to take a look at this too

You get the idea. Basically, it's meant to help foster transparency throughout the group, not just to put more eyes on a commit but to improve the development cycle for the team as a whole.

like image 76
Jay Avatar answered Nov 01 '22 06:11

Jay


Many of the things mentioned above are useful when getting notified of checkins. I typically use it for a couple of things:

  • If the project breaks in some way, sometimes I can more quickly hone in on the problem because I've been aware of the checkins that have been going into the system. It might point me to a solution more quickly
  • It provides an easily searchable set of checkins (at least because i receive them in email). Yes, your source control system has all that info, but it might not be easy to search the entire checkin comment. With email, it's pretty trivial for me to go back and search for a user, module, keyword, etc and have Outlook cough up the relevant emails.
  • I can easily keep track of younger developers and what they are doing. It gives me a chance to see when they are checking in code and what they are doing with the code. It gives ongoing chances to mentor outside of other regularly scheduled things like code reviews.
  • It provides a way for the team to track progress and note when they might be checking in conflicts.

Even if there are many notifications, I don't feel that they have to be read in their entirety. I skim them a bunch, but that's enough to get me back to the appropriate commits when I need more information.

like image 28
Mark Avatar answered Nov 01 '22 05:11

Mark


It can be useful if someone commits a change to one or more programs that you yourself may be working on. It's a heads up, if you will, that you may need to resolve some conflicts that may require further communication.

I've seen the situation where programmer A committed some changes and then went on vacation a couple of days later. Programmer B was also working on the same programs and there were some conflicts at the time Programmer B went to commit his changes. Normally this is no biggie, plus communication should always be kept open between the team members. In this case, Programmer B had some questions about the changes made by Programmer A, but had to wait a week until that programmer had returned. A heads up email, even an auto generated one, would have been helpful in this situation.

Just my two cents.

like image 41
Jason Down Avatar answered Nov 01 '22 06:11

Jason Down


Sure, you want to eyeball the code getting put in so you have some idea what other people in your team are doing, and you're not stuck in a cave just writing your code.

Even if you're not monitoring the code for the quality, you can have an idea what other people are working on.

It helps build the team.

like image 37
Andy Lester Avatar answered Nov 01 '22 06:11

Andy Lester