ocaml-sf / learn-ocaml-autogen Goto Github PK

When creating an exercise from an input file with missing metadata fields, the generated meta.json contains fields with value null. It is avoidable by not generating the meta file at all. Currently, an warning is printed. I think it is better to leave it as is, because building will fail anyway without a meta.json, but not with a null value inside a field that is not strictly mandatory (e.g authors).

In its current version, the program crashes when encountering a function with more than 4 arguments that is not a %prelude or %prepare function. It would be nice to overtake this limitation. There is a test function in Test_lib to do this.

Some annotations are not documented.

Travis tells us that autogen built is failing on Mac OS. Indeed, a Debian related command is issued.

There is no documentation for the compilation and installation protocols.

Please create a .travis.yml file to activate continuous integration.

Please follow a more standard organization for the source tree of this repository, i.e.:

• src : contains the source file of the tool.
• tests : contains tests and testing infrastructure (typically shell scripts).
• doc : contains development documentation and/or user manual.
• bin : contains the produced executable.