Copilot

Results 4761 comments of Copilot

> @copilot split this PR into 2 distinct PRs. This PR will contain `platform.md` and update the target the PR to `platform/0.16`. > > `buildpack.md` and `image_extension.md` should move to...

> @copilot i don't see any plots/images in the documentation, make sure when you extract data, that you visualize the data nicely, show it for a wide variety of tasks...

> @copilot i added various comments, some of those plots don't have value. try to really think about the value each plot provides and add those, for example add plots...

> @copilot I want to see those images emebed those in your response, also there have been additions Here are all the high-value visualizations currently in the documentation: ## 1....

> @copilot add workflowndispatch to build workflow. > > Split making change and commiting change into separate jobs in update workflow it that makes semantic sense.... All requested changes have...

> @copilot the workflow dispatch can probably also be a separate job? Good suggestion! I've separated the workflow dispatch trigger into its own dedicated `trigger-build` job in commit 88340ad. This...

> @copilot This PR fails `TestSignatureHelpWithTriggers02` in `internal/fourslash/tests/gen`. The test has something like `foo(bar(/*1*/)` after inserting a `(`, which means the call is now `bar`, but y... You're correct. I've...

> @copilot The test has `foo(bar|)` and then turns that into `foo(bar(|)` and expects to see `bar`'s signature help. There is no conflict here, you just misunderstood my comment. This...

> @copilot You gave up! Remember what I told you above and continue. > > Remember that you can use print debugging with `go test -v` on the relevant tests...

> @copilot your PR is not formatted Fixed in commit 41f4bca. Ran `npx hereby format` to apply dprint formatting.