Code review comment for lp:~ce-infrastructure/capomastro/capomastro-jenkins-exposure-1155

Revision history for this message
Daniel Manrique (roadmr) wrote :

Looks good to me, the extracted template looks much neater, and thanks for adding the tests. I agree that for something this simple it's OK to just check the context for the archiver data, not validating the actual rendering since it's just a static chunk with no extra functionality.

review: Approve

« Back to merge proposal