Code review comment for lp:~fginther/charms/trusty/jenkaas/configure-slaves
Revision history for this message
| Francis Ginther (fginther) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
> I'm not a fan of the InRelation and NotInRelation classes (since they are
> essentially a method call) but if that's the only way then so be it.
I dug some more on this and think I can see a way to do this inside the 'data_ready' list directly without creating the separate class. Let me see if I can get this to work while resolving the merge conflicts.