-
Notifications
You must be signed in to change notification settings - Fork 52
Code Style Guide
Cyrus Omar edited this page Sep 12, 2019
·
14 revisions
- Don't leave any warnings. If necessary, you can suppress a warning by putting the
[@warning "-#"]
annotation narrowly on the node generating the warning, where#
is the warning code. Leave an explanation for why that annotation is necessary when you do so.
- Each type should have its own module, even if it is small.
- Whenever possible, place
[@deriving sexp]
on type definitions, which will automatically derive sexp serializers. This will generally require anopen Sexplib.Std
at the top of your module to open the standard sexp serializers for base types likestring
andint
. - List the preferred variable name prefixes with a comment of the form
/* Variable: <preferred> */
before the type definition.
- Each module should have a corresponding signature (
.rei
) file if there are any top-level local definitions. - Local definitions should be fully type annotated.