Well, it seemed to say that it was both optional, and didn't say anything
about doing it when reproposing. Given the tracking of comment threads, I
would hope you wouldn't have to rebase after proposing.

John
=:->



On Mon, Jun 2, 2014 at 4:09 PM, roger peppe <[email protected]> wrote:

> On 2 June 2014 12:44, John Meinel <[email protected]> wrote:
> > Well the linked doc said to rebase before propose:
> >
> > When ready for feedback, push your feature branch to github, optionally
> > after
> > collapsing multiple commits into discrete changes:
> >
> > $ git rebase -i --autosquash master
> > $ git push origin new_feature
>
> I saw that, but that doesn't make it clear to me if one should
> rebase at every round of the review, or just when initially proposing.
>
> If possible, I hope we can still see the changes in each round
> of the review in the github history and that we can get
> to that review process easily from the commit history.
>
> I'm not entirely sure when github still shows discussion on commits
> that aren't in the history of a branch.
>
-- 
Juju-dev mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju-dev

Reply via email to