Skip to content
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

feat: Add VS Code launch.json config #20

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

teofanis
Copy link
Contributor

@teofanis teofanis commented Dec 5, 2022

This PR address Issue #18

Here's how it looks during setup :
image

If no is selected, there will be no changes, if yes is selected the following will be created within the user's aoc directory :
image
image

I decided to open the PR, so that we can comment on something, rather than just imagining it.
Configuration and wording is not final, so yes - I await your comments.

@teofanis
Copy link
Contributor Author

teofanis commented Dec 5, 2022

P.S - Just noticed I failed to mention VS Code in the question :D, anyways the question should be rephrased definitely

@terryaney
Copy link

terryaney commented Jan 3, 2024

Does this support debugging the daily index.ts files? I added following to my launch config just to be able to hit f5 on currently open file. Battled a few issues, but seems to be working pretty solidly now:

{
    "version": "0.2.0",
    "configurations": [
        {
            "type": "node",
            "request": "launch",
            "name": "Launch Program",
            "program": "${file}",
            "preLaunchTask": "tsc: build - tsconfig.json"
        }
    ]
}

@teofanis
Copy link
Contributor Author

teofanis commented Jan 3, 2024

I don't remember as I made the pr last year xD. I believe you could run with debug on in Vs code needs to be tested, I can try and pull it to test in anyway the snippet you provided lgtm also.

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