Module Name Plugin: Treat modules starting with lowercase as Main module #616
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently, module name plugin treats basename of file as module name.
This inferes, for example, the module name of
app/exe01.hs
asexe01
, contrary to the expectation that they must beMain
.This pull-requests addresses this issue by defaulting to
Main
when the file name is starting with non-upper cases.Perhaps, the logic itself can be refined so that it infers it
Main
only when the lowercased file is located right after the source-dirs?