Code review comment for lp:~mwhudson/launchpad/code-publisher-in-lp.code

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

Hi, can you look at this again? It failed tests so I made a few more changes.

I think we could eliminate some boilerplate by writing our own ZCML directive, how would something like

<lp:site
   name="code"
   web_service="true"
   request_class="lp.code.publisher.CodeLayer"/>

look to you?

« Back to merge proposal