Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Detect unused module declarations #684

Merged
merged 3 commits into from Jul 19, 2016

Conversation

Projects
None yet
3 participants
@alainfrisch
Copy link
Contributor

commented Jul 12, 2016

Updated version of #158

@UnixJunkie

This comment has been minimized.

Copy link
Contributor

commented Jul 18, 2016

Thanks Alain for following this issue.

@alainfrisch alainfrisch force-pushed the alainfrisch:unused_module_warning branch from 99f9455 to cc362cf Jul 18, 2016

@alainfrisch alainfrisch force-pushed the alainfrisch:unused_module_warning branch from cc362cf to 8c00e80 Jul 18, 2016

@alainfrisch

This comment has been minimized.

Copy link
Contributor Author

commented Jul 18, 2016

Will merge soon if nobody is against.

@alainfrisch alainfrisch merged commit 5adb809 into ocaml:trunk Jul 19, 2016

0 of 2 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@@ -19,7 +19,7 @@ OCaml 4.04.0:
(Florian Angeletti)


### Compiler user-interface:
### Compiler user-interfaceand warnings:

This comment has been minimized.

Copy link
@samoht

samoht Jul 19, 2016

Member

missing space?

camlspotter pushed a commit to camlspotter/ocaml that referenced this pull request Oct 17, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.