-
Notifications
You must be signed in to change notification settings - Fork 13.5k
lldb-dap extension CI tests #138932
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
Comments
@llvm/issue-subscribers-lldb-dap Author: John Harrison (ashgti)
As we add more support to the vscode lldb-dap extension, it would be good to create some standard tests and CI checks for the vscode extension code.
Today, we're not checking things like if the package.json is valid or that |
Its worth noting that there are multiple VSCode extensions in the llvm project.
Some consistency between these three may be helpful. |
is any of the other extensions currently tested in CI ?, as it would be easier to adapt it to the dap extension. as I am not sure if the build dependencies such as npm/typescipt are in the build bot machines. |
As we add more support to the vscode lldb-dap extension, it would be good to create some standard tests and CI checks for the vscode extension code.
Today, we're not checking things like if the package.json is valid or that
npm run package
is working as expected. They were broken recently by a bad merge and it was not caught by any CI jobs.The text was updated successfully, but these errors were encountered: