Code review comment for lp:~kevin-wright-1/u1db-qt/synchronizer-merged-with-trunk-8-aug

Revision history for this message
Cris Dywan (kalikiana) wrote :

> Den 09/08/2013 11:20, skrev Christian Dywan:
> > These are the doc errors I'm currently seeing with the latest changes. They
> need to be resolve before the branch can be merged:
> Hopefully all fixed now. Could not find anything specific to resolve the
> warnings about \property, but maybe this is resolved by the change to
> fix the warning about \class.
>
> Also \param is used elsewhere in source files, but perhaps was not used
> properly in these instances. The references have been removed in the
> latest commit, but perhaps this needs to be looked at again.

There's still 17 qdoc warnings. See https://jenkins.qa.ubuntu.com/job/u1db-qt-quantal-amd64-ci/7/console I locally see the same.

review: Needs Fixing

« Back to merge proposal