Skip to content

Typecheck main, fix #1333#1350

Closed
paf31 wants to merge 1 commit into
masterfrom
1333
Closed

Typecheck main, fix #1333#1350
paf31 wants to merge 1 commit into
masterfrom
1333

Conversation

@paf31

@paf31 paf31 commented Aug 9, 2015

Copy link
Copy Markdown
Contributor

@garyb What do you think? Anything with the name main is now typechecked against the type Eff _ _.

Maybe we could add an option --no-main-check in case this is too strict?

@garyb

garyb commented Aug 9, 2015

Copy link
Copy Markdown
Member

Anything with the name main? I think that could well be a bit too strict... a tricky problem now given that the main argument is from psc-bundle I guess. It seems like we should do the check there somehow, but that's not really possible either. Can we glean some clue from the JS as to whether it might be Eff instead? Not reliably I suppose. Argh.

@paf31 paf31 closed this Aug 23, 2015
@paf31 paf31 deleted the 1333 branch September 14, 2015 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants