fix: hack to workaround dune bug #1213
Merged
Conversation
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
Signed-off-by: Rudi Grinberg <me@rgrinberg.com> <!-- ps-id: b08aa88f-6b24-4fef-a04e-e5ef6f8171db -->
Pull Request Test Coverage Report for Build 4031
💛 - Coveralls |
voodoos
added a commit
to voodoos/ocaml-lsp
that referenced
this pull request
Jan 29, 2024
commit ccedef9 Author: Ulysse Gérard <thevoodoos@gmail.com> Date: Wed Jan 3 17:42:08 2024 +0100 fmt commit 1c3ebfd Author: Ulysse Gérard <thevoodoos@gmail.com> Date: Wed Jan 3 17:41:52 2024 +0100 rename: support project-wide but unsafe renaming commit 7105c12 Author: Ulysse Gérard <thevoodoos@gmail.com> Date: Thu Nov 30 16:22:28 2023 +0100 wip: make rename project wide commit ba1d99a Author: Ulysse Gérard <thevoodoos@gmail.com> Date: Thu Nov 30 14:49:28 2023 +0100 Notify the user when the index is out of date commit ef50a83 Author: Ulysse Gérard <thevoodoos@gmail.com> Date: Wed Nov 29 19:00:28 2023 +0100 wip: get desync info from merlin commit 5265d22 Author: Ulysse Gérard <thevoodoos@gmail.com> Date: Wed Nov 22 11:12:48 2023 +0100 Handle new `unit_name` config directive commit 2367339 Author: Ulysse Gérard <thevoodoos@gmail.com> Date: Thu Oct 26 15:50:49 2023 +0200 Fix merlin config merging commit 55ab533 Author: Ulysse Gérard <thevoodoos@gmail.com> Date: Wed Oct 25 16:33:38 2023 +0200 config: accept index_file directive commit e353220 Author: Ulysse Gérard <thevoodoos@gmail.com> Date: Fri Sep 29 14:23:32 2023 +0200 Apply compiler libs changes commit 69fea3a Author: Ulysse Gérard <thevoodoos@gmail.com> Date: Tue Jun 13 17:42:00 2023 +0200 Support project wide occurrences commit d3d8de5 Author: Ulysse <5031221+voodoos@users.noreply.github.com> Date: Mon Dec 18 16:34:42 2023 +0100 Prepare release 1.17.0 (ocaml#1219) * chore: restore lost version number in changes (erased in c8c1096) * chore: bump version number in changes before release commit f9de892 Author: Ulugbek Abdullaev <ulugbekna@gmail.com> Date: Mon Dec 11 19:51:35 2023 +0100 Completion for `in` keyword (ocaml#1217) feat(auto-completion): auto-complete `in` (in a hacky way) in .ml files commit 9dc8de2 Author: Jules Aguillon <jules@j3s.fr> Date: Mon Nov 27 04:29:17 2023 +0100 Upgrade OCamlformat to 0.26.1 (ocaml#1177) commit 2ffaea0 Author: Rudi Grinberg <me@rgrinberg.com> Date: Sun Nov 26 20:29:02 2023 -0600 chore: move dune-release to own shell (ocaml#1215) Signed-off-by: Rudi Grinberg <me@rgrinberg.com> commit 17eec24 Author: Rudi Grinberg <me@rgrinberg.com> Date: Sun Nov 26 20:07:08 2023 -0600 chore: update flakes (ocaml#1214) Signed-off-by: Rudi Grinberg <me@rgrinberg.com> commit 73bf594 Author: Rudi Grinberg <me@rgrinberg.com> Date: Sun Nov 26 19:51:42 2023 -0600 Completion edge cases (ocaml#1212) * Tests passing * Added support for whitespace in completion The solution here is to change all whitespace to spaces for ease of regex matching(all whitespace is equivelent semantically) and then remove all spaces from the prefix that's passed to merlin. Co-authored-by: faldor20 <eli.jambu@gmail.com> Co-authored-by: faldor20 <eli.jambu@yahoo.com> commit 3b3b9bc Author: Rudi Grinberg <me@rgrinberg.com> Date: Sun Nov 26 18:58:33 2023 -0600 fix: hack to workaround dune bug (ocaml#1213) Signed-off-by: Rudi Grinberg <me@rgrinberg.com> commit c6eafd8 Author: panglesd <peada@free.fr> Date: Sun Oct 22 09:32:24 2023 +0200 Compatibility with odoc-parser.2.3.0 (ocaml#1184) * compatibility with odoc-parser.2.3.0 Signed-off-by: Paul-Elliot <peada@free.fr> commit 6f78852 Author: Rudi Grinberg <me@rgrinberg.com> Date: Tue Oct 17 23:36:08 2023 -0600 fix(test): correctly qualify pp dep (ocaml#1203) Signed-off-by: Rudi Grinberg <me@rgrinberg.com> commit 6bda990 Author: Bao Zhiyuan <71200607+bzy-debug@users.noreply.github.com> Date: Wed Oct 18 13:35:39 2023 +0800 Fix uri to_string (ocaml#1197) The current implementation is not the same as uri.ts see microsoft/vscode-uri@96acdc0/src/uri.ts#L601 the ts code just checks if authority is empty, while ocaml code checks if authority is "file" Co-authored-by: Bao Zhiyuan <baozhiyuan@idea.edu.cn> commit e0c4df3 Author: Rudi Grinberg <me@rgrinberg.com> Date: Mon Oct 16 22:36:44 2023 -0600 fix: odoc-parser constraints (ocaml#1204) we aren't compatible with 2.3.0 Signed-off-by: Rudi Grinberg <me@rgrinberg.com> commit eca77cf Author: Rudi Grinberg <me@rgrinberg.com> Date: Mon Oct 16 14:58:31 2023 -0600 chore: more CHANGES formatting (ocaml#1202) Signed-off-by: Rudi Grinberg <me@rgrinberg.com> commit 208683e Author: Rudi Grinberg <me@rgrinberg.com> Date: Mon Oct 16 12:25:13 2023 -0600 chore: better formatting in CHANGES (ocaml#1201) Signed-off-by: Rudi Grinberg <me@rgrinberg.com> commit ce2e46b Author: Rudi Grinberg <me@rgrinberg.com> Date: Mon Oct 16 11:21:19 2023 -0600 chore: update flakes (ocaml#1195) Signed-off-by: Rudi Grinberg <me@rgrinberg.com> commit 841fe5c Author: Rudi Grinberg <me@rgrinberg.com> Date: Mon Oct 16 11:16:05 2023 -0600 refactor: fix against newer versions of dune (ocaml#1200) removes the incomplete pattern match warning on the progress Signed-off-by: Rudi Grinberg <me@rgrinberg.com> commit 672ca1b Author: Ulysse <5031221+voodoos@users.noreply.github.com> Date: Wed Oct 4 00:14:05 2023 +0200 Only return first result for type_enclosing. (ocaml#1196) in `type-annotate` code action.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Signed-off-by: Rudi Grinberg me@rgrinberg.com