forked from ocaml/merlin
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Promote fixed test after Dune upgrade
- Loading branch information
Showing
1 changed file
with
9 additions
and
4 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,12 +1,17 @@ | ||
$ dune build @check | ||
|
||
FIXME: When the deifinition is in one of the implicit transitive dependencies | ||
Merlin does not found the file in the source path provided by Dune. One possible | ||
fix would be for Dune to provide additional source path for "externatl" deps. | ||
When the deifinition is in one of the implicit transitive dependencies Merlin | ||
used to not find the file in the source path provided by Dune. | ||
$ $MERLIN single locate -look-for ml -position 1:15 \ | ||
> -filename bin/main.ml <bin/main.ml | ||
{ | ||
"class": "return", | ||
"value": "'Lib1.t' seems to originate from 'Lib2' whose ML file could not be found", | ||
"value": { | ||
"file": "$TESTCASE_ROOT/src/lib2/lib2.ml", | ||
"pos": { | ||
"line": 1, | ||
"col": 0 | ||
} | ||
}, | ||
"notifications": [] | ||
} |