Hi, new guy here. I am currently looking at a good...
# contribute
l
Hi, new guy here. I am currently looking at a good-first-issue issue that is related to the cmd. I have implemented the change, I would like to test my change functionally locally before creating a PR, however, I am getting a lot of errors for attempting to build it using the command
make dist
. Normally what would be the best approach to test functionality changes? Would it be local or directly on the CI/CD pipeline? WOuld love to know the best approach here.
m
It would be good to get your local setup working. The CI pipeline can only be run by Pulumi employees. We’re happy to do that for you once you have a PR, but to get it ready it’s better to test locally. What error are you seeing?
l
@melodic-tomato-39005 I am trying to debug the pulumi cli using VSCode’s debugger for the issue of #13542. The launch.json is as following and I have placed breakpoints at /pkg/cmd/pulumi/main.go and /pkg/cmd/pulumi/about.go. However, the breakpoint is not showing. What could be the right wya to debug the pulumi cli
{
"version": "0.2.0",
"configurations": [
{
"name": "Launch Package",
"type": "go",
"request": "launch",
"mode": "debug",
"program": "/Users/tzehaotay/Desktop/TzeHao/pulumi/pkg/cmd/pulumi/main.go",//"${fileDirname}",
"args": ["about"]
"cwd": "/Users/tzehaotay/Desktop/TzeHao/entrypoint-test/pulumi"
}
]
}
h
I can’t see anything obviously wrong in there - here’s what I use that works:
Copy code
{
  "version": "0.2.0",
  "configurations": [
    {
      "name": "Debug pulumi",
      "type": "go",
      "request": "launch",
      "mode": "auto",
      "program": "${workspaceFolder}/pkg/cmd/pulumi",
      "cwd": "${workspaceFolder}/../test-program",
      "args": [
        "about"
      ]
    }
  ]
}
l
Seems like it doesn’t work when the launch.json is not in the pulumi workspace. I tried running the launch.json in the entrypoint-test folder and pulumi folder. It only worked in the latter even though the launch.json are the same (given that the path are adjusted accordingly)