Re: ALTO docs for Lithium

Home Messages Hashtags Subgroups

This group is locked. No changes can be made to the group while it is locked.

#105

Re: ALTO docs for Lithium

Colin Dixon <colin@...> #105 A few points: 1.) It seems like 20762 depends on 20475 (patch set 2). 2.) It seems like there are edits present in 21600 that aren't in the combination of 20762 and 20475—specifically removing the getting started guide content. Based on the assumption that 21600 was the closest to what you really wanted I copied that and replaced the changeId so that I could resubmit it as Patch Set 4 of 20762. Let me know if that looks wrong. In the meantime, I'm going to assume that this is the right document to review and I'll likely get to that soon: https://git.opendaylight.org/gerrit/#/c/20762/4 --Colin toggle quoted message Show quoted text On Wed, Jun 3, 2015 at 10:25 PM, Gao Kai <godrickk@...> wrote: 20475 has been abandoned. Sorry for the mess. On 04/06/15 03:08, Colin Dixon wrote: As far as I can tell there have been four submitted patches for ALTO docs: https://git.opendaylight.org/gerrit/#/c/18389/ https://git.opendaylight.org/gerrit/#/c/20475/ https://git.opendaylight.org/gerrit/#/c/20762/ https://git.opendaylight.org/gerrit/#/c/21600/ Patches 18389 and 21600 are both abandoned, but patches 20475 and 20762 are still active. It's really hard for me to tell which patches are redundant and even harder for me to track if the feedback has been applied already. Can you help me understand which of these is still relevant? In the future, you really want to respond to a review with a new patch set on the same gerrit change, not a new change altogether. Instructions on how to do that are here: https://wiki.opendaylight.org/view/Documentation/Tools#Updating_an_Existing_Patch --Colin

View All 9 Messages In Topic

#105

Join z.archive.alto-dev@lists.opendaylight.org to automatically receive all group messages.

Home Hashtags Subgroups Terms