Reminder: juju-core github migration
John Meinel
john at arbash-meinel.com
Mon Jun 2 13:14:13 UTC 2014
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 <rogpeppe at gmail.com> wrote:
> On 2 June 2014 12:44, John Meinel <john at arbash-meinel.com> 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.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/juju-dev/attachments/20140602/b9997813/attachment.html>
More information about the Juju-dev
mailing list