-
Notifications
You must be signed in to change notification settings - Fork 97
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
Failed to resolve module path identifier #1083
Comments
Can you please see if applying PR #1081 helps at all? It looks to me like it's looking up a shadowed value which doesn't work well in master - that PR might fix it. |
I don't see the warning anymore with this patch indeed! |
Good news, thanks for checking! |
@jonludlam I believe this can be closed, right ? :) |
Yes :-) |
@jonludlam, I tried again now that a new version has been released since and it seems to be back:
I'm on odoc |
(I can't re-open the issue...) |
It's not fixed in v2.4.x, it's in 3.0 (and there's only a beta of that out yet!) |
Hi,
When applying this change to Owi, I got the following:
This may be similar to #789 but I'm not completely sure.
The text was updated successfully, but these errors were encountered: