Page MenuHomePhorge

Use Amnesia, get rid of Ecto
ClosedPublic

Authored by machniak on Jan 18 2017, 2:03 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 16, 11:30 PM
Unknown Object (File)
Sat, Apr 13, 10:31 AM
Unknown Object (File)
Tue, Apr 9, 10:33 AM
Unknown Object (File)
Mon, Apr 8, 9:33 AM
Unknown Object (File)
Sun, Apr 7, 8:44 PM
Unknown Object (File)
Sat, Apr 6, 8:19 AM
Unknown Object (File)
Mar 7 2024, 3:03 AM
Unknown Object (File)
Feb 20 2024, 2:51 PM
Subscribers

Details

Summary

@seigo, I'm not sure about these "require Amnesia" statements

Diff Detail

Repository
rKC kolab-chat
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

machniak retitled this revision from to Use Amnesia, get rid of Ecto.
machniak updated this object.
machniak edited the test plan for this revision. (Show Details)
machniak added a reviewer: Kolab Chat Developers.
machniak added a subscriber: seigo.
seigo added a reviewer: seigo.

Those require statements look fine IMHO. Yes, it's a bit more verbose that one could imagine if it were placed somewhere central, but at least this way it is explicit: this code requires that API .. and so it says so right there. That kind of clarity makes that extra line worth it ... +1 from me.

This revision is now accepted and ready to land.Jan 20 2017, 10:55 AM
This revision was automatically updated to reflect the committed changes.