Copilot

Results 4702 comments of Copilot

@blocher 👋 This repository doesn't have [Copilot instructions](https://docs.github.com/enterprise-cloud@latest/copilot/how-tos/configure-custom-instructions/add-repository-instructions?tool=webui). With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs. I can generate a .github/copilot-instructions.md file...

@tanyasethi-msft I've opened a new pull request, #25780, to work on those changes. Once the pull request is ready, I'll request review from you.

@tanyasethi-msft I've opened a new pull request, #25786, to work on those changes. Once the pull request is ready, I'll request review from you.

@tanyasethi-msft I've opened a new pull request, #25794, to work on those changes. Once the pull request is ready, I'll request review from you.

> @copilot please merge code from `main` branch and resolve conflicts. Merged main branch and resolved conflicts. The merge integrates the better QRCodeDetector/QRCodeDetectorAruco definitions from main while preserving the QRCodeEncoder...

> @copilot please merge code from main branch and resolve conflicts. Merged main branch (commit 8b6120a) and resolved conflicts. The merge incorporates the boxPoints type fix from PR #97 while...

> @copilot Where are the tests? > > Also it looks like the PR may need to merge from upstream and run `hereby generate` again. Added test in `TestCompletionJSDocBeforePropertyNoCrash` that...

> @copilot try to address. I'll address this by adding special handling for `OpenBracketToken` in the language service layer. The fix will handle completions after `[` in array literals by...

> > Why does this need to differ so much from the original one in Strada? > > Because in JavaScript, indexing into `-1` would just get you `undefined` which...

> @copilot I think the better thing to do might actually be to get rid of `registerMultiProjectReferenceRequestHandler`, and shift more of that work into a common helper that `handleReferences` and...