Skip to content

Add test for issue #1575 Completion for OCaml object methods#1576

Closed
sidkshatriya wants to merge 1 commit intoocaml:masterfrom
sidkshatriya:issue-1575
Closed

Add test for issue #1575 Completion for OCaml object methods#1576
sidkshatriya wants to merge 1 commit intoocaml:masterfrom
sidkshatriya:issue-1575

Conversation

@sidkshatriya
Copy link
Copy Markdown
Contributor

See #1575

@voodoos
Copy link
Copy Markdown
Collaborator

voodoos commented Mar 6, 2023

Thanks for the test !

@voodoos
Copy link
Copy Markdown
Collaborator

voodoos commented Mar 6, 2023

If you are curious and have some tome to investigate yourself here are a few pointers:

  • You can add -log-file - in your test command to print enable logging to stderr
  • You can start investigating at Complete_prefix in the Query_commands module line 403
  • dune build @issue1575 --watch should give you some useful live feedback

Anyway I will probably have a look next week.

@voodoos
Copy link
Copy Markdown
Collaborator

voodoos commented May 16, 2023

Closing. This test is now part of another PR with a fix: #1606

@voodoos voodoos closed this May 16, 2023
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