Code review comment for lp://staging/~mhr3/dockmanager/metadata-update

Revision history for this message
Robert Dyer (psybers) wrote :

First of all, your diff includes way more than what you wanted. You should fix that up.

Second, what is the point to the metadata changes? What is the format used? Is it just a true INI file format? Must we verify the section is named 'DEFAULT'? What if it isnt, do we still use it? Will we allow other sections, if so what are those?

review: Needs Fixing

« Back to merge proposal