Code review comment for lp:~wgrant/launchpad/destroy-distroseries-lucilleconfig

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

Looks good, thanks for working on this.

Did you have a pre-implementation call with anybody about this?

Just one minor comment - can you clarify the comment on enabled_architectures to say more than just that it contains the enabled architectures? As we had a property earlier with the same name but different contents I think that would help prevent some potential confusion.

Do you need me to land this branch?

review: Approve (code)

« Back to merge proposal