Depth and reset #20
Depth and reset #20
Conversation
|
Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests |
|
Any review of this pull request? We've been using this code in our development at $DAYJOB for a few months with no issues. |
| @@ -0,0 +1,5 @@ | |||
| <div> | |||
| <p> | |||
| When this is checked the first thing to do will be a <pre>repo forall -c "git reset --hard"</pre> | |||
There was a problem hiding this comment.
Doesn't repo sync -d accomplish the same?
There was a problem hiding this comment.
no, not quite the same. both "repo sync" and "repo sync -d" will report an error if local uncommitted changes would be lost in the sync.
|
Well, it looks good to me. But @aep has done a few releases not long ago, so I'd prefer to wait a bit longer to see if he responds. So I don't step on anyone's toes by merging. |
|
Looks good to me. @rsandell i'd be delighted if you could step in actually. I changed dayjobs and won't be able to spend much time with jenkins for a few months unfortunately. |
I've added two features for this plugin: support for shallow clones, option to reset the repo before syncing.