On Tue, 2017-06-06 at 21:40 +0200, Kamil Paral wrote:
> Thoughts? Objections to putting it to production?

Overall, I think this is fine, nice job. I do kinda hate that you have
to edit 12 separate test cases one-by-one if you make any changes to
the underlying structure (that was the purpose of the single top-level
template before), but the tradeoff of less nesting complexity is
probably worth it. Other than that most of the changes are clear
improvements, so thanks for that.
-- 
Adam Williamson
Fedora QA Community Monkey
IRC: adamw | Twitter: AdamW_Fedora | XMPP: adamw AT happyassassin . net
http://www.happyassassin.net
_______________________________________________
test mailing list -- test@lists.fedoraproject.org
To unsubscribe send an email to test-le...@lists.fedoraproject.org

Reply via email to