Skip to content

vscode debugger config #2554

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

Merged
merged 1 commit into from
Jul 26, 2024
Merged

Conversation

EmileTrotignon
Copy link
Collaborator

This adds config for the debuger provided by the earlybird vscode extension.
There might be some issues with enabling bytecode mode, but I do not know of any.

Its very very useful if you use vscode, I have managed to understand behaviours that were hard to debug-print with this.

@EmileTrotignon EmileTrotignon added the no changelog set this to bypass the CI check for changelog entries label Jul 1, 2024
"name": "OCaml earlybird (experimental)",
"type": "ocaml.earlybird",
"request": "launch",
"program": "${workspaceRoot}/_build/default/bin/ocamlformat/main.bc",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be great if this called dune build _build/default/bin/ocamlformat/main.bc before every runs.

@EmileTrotignon
Copy link
Collaborator Author

I would like this to be merged without extra work because it works, and is useful as is, and I need it. I would be happy to do the research necessary to also build the project through an vscode action, but I remember I tried when I first wrote this, and it did not work, so I think its not going to be a 10min job.

Copy link
Collaborator

@Julow Julow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, let's merge.

@Julow Julow merged commit 8421be1 into ocaml-ppx:main Jul 26, 2024
2 checks passed
@Julow
Copy link
Collaborator

Julow commented Dec 2, 2024

This was partly reverted in #2637 due to problem during the release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog set this to bypass the CI check for changelog entries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants