How to deal with important modifications
Hi,
I am working on the synchronization view. It is not 100% functional yet. The view is not updated when a local file is modified for example. As the modifications are getting important, I was wondering how to deal with it. Should I continue my work an send all the patches when finished?
To have an overview of the modifications:
http://github.com/yanns/egit/commit/18c4a928d53345802a8c9641dcb2d457ebbe2cbc
http://github.com/yanns/egit/commit/9fab398fa1b7b6efa9532b3c09e5bcfcc8bb9419
Or should I begin to send patches, but by not activating the function yet?
(It could be a way to have other people to help contributing.)
Yann
Quoted 12 lines · Yann Simon
Hi,
I am working on the synchronization view. It is not 100% functional yet. The view is not updated when a local file is modified for example. As the modifications are getting important, I was wondering how to deal with it. Should I continue my work an send all the patches when finished?
To have an overview of the modifications:
http://github.com/yanns/egit/commit/18c4a928d53345802a8c9641dcb2d457ebbe2cbc
http://github.com/yanns/egit/commit/9fab398fa1b7b6efa9532b3c09e5bcfcc8bb9419
Quoted 5 lines
Or should I begin to send patches, but by not activating the function yet?
(It could be a way to have other people to help contributing.)Yann
Yann,
I was asking myself the same questions about my work on ignores and chose to send it out early, being half completed. Don't know if that was right, did not receive feedback yet, but it's only been 2 days with Eclipsecon wrapping up on friday.
If you keep a seperate changeset in which you activate your work and split up the changesets in manageable pieces it's easier for others to review your work and comment on it.
Love to see your work though. Having the sync view available for git would be a major plus. I proposed something simpler on the wiki: a 'pending changes' window. But if you have the complete sync view that's wonderful.
my 2 cents :-)
Ferry
lördag 28 mars 2009 23:44:49 skrev "Ferry Huberts (Pelagic)" <[email protected]>:
Quoted 17 lines
Quoted 16 lines · Yann Simon
Hi,
I am working on the synchronization view. It is not 100% functional yet. The view is not updated when a local file is modified for example. As the modifications are getting important, I was wondering how to deal with it. Should I continue my work an send all the patches when finished?
To have an overview of the modifications:
http://github.com/yanns/egit/commit/18c4a928d53345802a8c9641dcb2d457ebbe2cbc
http://github.com/yanns/egit/commit/9fab398fa1b7b6efa9532b3c09e5bcfcc8bb9419Or should I begin to send patches, but by not activating the function yet?
(It could be a way to have other people to help contributing.)
Quoted 7 lines
Quoted 1 line
Yann
Yann,
I was asking myself the same questions about my work on ignores and chose to send it out early, being half completed. Don't know if that was right, did not receive feedback yet, but it's only been 2 days with Eclipsecon wrapping up on friday.
That's not the reason you haven't received a response. Basically, the larger a set of patches is, more time is needed.
Quoted 3 lines
If you keep a seperate changeset in which you activate your work and split up the changesets in manageable pieces it's easier for others to review your work and comment on it.
Indeed. Small patches can be reviewed more quickly if they introduce well defined changes and especially good is if they make sense of their own.
Think about how you would like the changes presented if you were to review them without knowing anything in advance.
-- robin
Quoted 9 lines · Robin Rosenberg
l?rdag 28 mars 2009 23:44:49 skrev "Ferry Huberts (Pelagic)" <[email protected]>:
Quoted 5 lines
I was asking myself the same questions about my work on ignores and chose to send it out early, being half completed. Don't know if that was right, did not receive feedback yet, but it's only been 2 days with Eclipsecon wrapping up on friday.
That's not the reason you haven't received a response. Basically, the larger a set of patches is, more time is needed.
Yup.
I plan to block off time to look at the patches tomorrow.
The bigger it is, the more likely it gets pushed off a few days, until I can block off a seemingly large enough amount of time to give it a complete reading.
Signature · 1 line
Shawn.