[PATCH 6 of 9 V6] bundle2: add `bookmarks` part handler

Pierre-Yves David pierre-yves.david at ens-lyon.org
Thu Nov 10 12:39:03 EST 2016



On 11/02/2016 10:56 AM, Stanislau Hlebik wrote:
>
>
> On 10/14/16, 2:28 AM, "Pierre-Yves David" <pierre-yves.david at ens-lyon.org> wrote:
>
>     What is your plan regarding hooks execution here?
>
> I planned to add it when I’m going to work on push
>
>     How do you handle cases where the part exist multiple time in the bundle ?
>
> Is there a problem with multiple parts? `op.records.add('bookmarks', bookmarks)` appends bookmarks from each part so it should work fine.

Yes but it adds them as multiple items instead of a unified dictionnary, 
this seems strange. Could we have them as a single dict instead?

-- 
Pierre-Yves David


More information about the Mercurial-devel mailing list