Account Options

  1. Sign in
The old Google Groups will be going away soon, but your browser is incompatible with the new version.
Google Groups Home
« Groups Home
Code Review process
There are currently too many topics in this group that display first. To make this topic appear first, remove this option from another topic.
There was an error processing your request. Please try again.
flag
  5 messages - Collapse all  -  Translate all to Translated (View all originals)
The group you are posting to is a Usenet group. Messages posted to this group will make your email address visible to anyone on the Internet.
Your reply message has not been sent.
Your post was successful
 
From:
To:
Cc:
Followup To:
Add Cc | Add Followup-to | Edit Subject
Subject:
Validation:
For verification purposes please type the characters you see in the picture below or the numbers you hear by clicking the accessibility icon. Listen and type the numbers you hear
 
Jay Beavers  
View profile  
 More options Aug 12 2009, 11:25 am
From: Jay Beavers <j...@hikinghomeschoolers.org>
Date: Wed, 12 Aug 2009 08:25:44 -0700
Local: Wed, Aug 12 2009 11:25 am
Subject: Code Review process

I have a patch I'd like to commit to the depot.  It includes adding the
xUnit 1.5 binaries and adding a library engine dll.
I attempted to follow the "submit an issue of template 'code review'"
process but this didn't work very well because the issue tracker insisted I
"include the branch path".  This confused me because we don't have 'branch
paths' in HG projects to the best of my understanding, I think this template
assumes a subversion-like approach to source management.

I exported my changes using hg bundle (have have 2 commits, a simple hg
export didn't seem to be the right approach).  I would have attached the
file here but it's 1.6 MB since it includes the xUnit binaries and I didn't
want to spam everyone's inbox with a large attachment.

Suggestions on how to proceed?  I'm working on setting up my dev box to be
http visible so I can "hg serve" my repository, but I'm having to go through
a bit of mess due to dynamic ip, nats, etc.


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Jay Beavers  
View profile  
 More options Aug 12 2009, 11:40 am
From: Jay Beavers <j...@hikinghomeschoolers.org>
Date: Wed, 12 Aug 2009 08:40:02 -0700
Local: Wed, Aug 12 2009 11:40 am
Subject: Re: Code Review process

On the off chance that everything is working, my hg repository is now served
via http://hikingforum.homedns.org:8000.  Hopefully this doesn't mean you
can all now read my bank account information :-)
I'm submitting changesets 17 & 18 for review.

On Wed, Aug 12, 2009 at 8:25 AM, Jay Beavers <j...@hikinghomeschoolers.org>wrote:


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Noah Hart  
View profile  
 More options Aug 12 2009, 11:49 am
From: Noah Hart <noah.h...@gmail.com>
Date: Wed, 12 Aug 2009 08:49:31 -0700
Local: Wed, Aug 12 2009 11:49 am
Subject: Re: Code Review process
Miguel

Do you have experience with your mono projects about how best to add
multiple development paths here?

Noah


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Miguel de Icaza  
View profile  
 More options Aug 13 2009, 11:42 am
From: Miguel de Icaza <miguel.de.ic...@gmail.com>
Date: Thu, 13 Aug 2009 11:42:11 -0400
Local: Thurs, Aug 13 2009 11:42 am
Subject: Re: Code Review process

Hello Noah,

Do you have experience with your mono projects about how best to add

> multiple development paths here?

Many of our developers keep private trees and forks while they develop a
feature, but at the end of the day, we need to merge all of it in the
central repository.

The central repository is where the official release comes from, where the
testing is done, where the packaging is done, so there is always a strong
incentive to get the patches back up.

Miguel.


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Phil Haack  
View profile  
 More options Aug 13 2009, 12:15 pm
From: Phil Haack <phi...@microsoft.com>
Date: Thu, 13 Aug 2009 09:15:06 -0700
Local: Thurs, Aug 13 2009 12:15 pm
Subject: RE: Code Review process

We follow the same process with Subtext, though we use private branches rather than private repositories since we're still using positively archaic centralized source control (Subversion), not like you cutting edge distributed source control people. ;) The concept and process is the same though.

In general, we avoid doing any half-ass checkins to the main repository. We also have a continuous integration server which is publicly available: http://build.subtextproject.com/ccnet/ViewFarmReport.aspx

If you go there right now, you'll even see that our trunk is broken due to a 32bit/64bit issue I'm in the middle of fixing. Bad Phil! Bad! :) I think a CI server is an absolute necessity.

Phil

From: csharp-sqlite@googlegroups.com [mailto:csharp-sqlite@googlegroups.com] On Behalf Of Miguel de Icaza
Sent: Thursday, August 13, 2009 8:42 AM
To: Noah Hart
Cc: csharp-sqlite@googlegroups.com
Subject: Re: Code Review process

Hello Noah,
Do you have experience with your mono projects about how best to add multiple development paths here?

Many of our developers keep private trees and forks while they develop a feature, but at the end of the day, we need to merge all of it in the central repository.

The central repository is where the official release comes from, where the testing is done, where the packaging is done, so there is always a strong incentive to get the patches back up.

Miguel.


 
You must Sign in before you can post messages.
To post a message you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
End of messages
« Back to Discussions « Newer topic     Older topic »