| kata-irc-bot | <bergwolf> IMO we don't need to persist in container.delete() method. | 02:15 |
|---|---|---|
| kata-irc-bot | <bergwolf> In fact, I think the entire container.delete() method should be dropped. It calls sandbox.removeContainer to remove itself which is a layering violation imho. The only meaningful logic in c.delete() is the container state check, which should be moved to a concrete method to properly define and handle container state transition. Then we can open code sandbox.removeContainer in sandbox.go | 02:18 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!