lukaszcz / coqhammer

CoqHammer: An Automated Reasoning Hammer Tool for Coq - Proof Automation for Dependent Type Theory
Other
211 stars 31 forks source link

Adjust Dune build flags and add tests via Dune for 8.12 #87

Closed palmskog closed 3 years ago

palmskog commented 3 years ago

Here are some adjustments to Dune builds to follow best practices. I also added tests corresponding to test-plugin and test-tactics. Regular Make builds are untouched.

Note that Dune does not allow running tests separately unless the files are located in different directories (unlike tests/tactics_test.v and tests/plugin_test.v). So maybe the tests directory could be partitioned into tests/tactics and tests/plugin? I can add this to the PR if such a change would be welcome.

lukaszcz commented 3 years ago

So maybe the tests directory could be partitioned into tests/tactics and tests/plugin? I can add this to the PR if such a change would be welcome.

Yes, this change would be welcome.