Copilot
Copilot
> While the implementation correctly handles the strictness of closing delimiters (using `==` to ensure longer quote sequences are treated as content), there are significant issues regarding brace escap... Working...
> > **Status:** 598/601 tests passing. The brace detection logic is now correct, but token boundary emission needs debugging. The 3 failing tests show that while we correctly identify escaped...
> The current implementation of interpolated raw string literals in [`Scanner.cs`] has a bug in how it determines when braces open an interpolation hole versus when they remain as literal...
> Add the test cases suggested in the [previous review](https://github.com/atifaziz/CSharpMinifier/pull/56#pullrequestreview-3647033594). Make sure they all pass. If the test cases are already covered t... Looking at the current test suite, I...
> > Will fix the implementation first, then add any missing test cases. > > Proceed Working on fixing the brace counting implementation. The current code has issues with: 1....
@matthiask 👋 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...
> @copilot this fails with the following error: > > ``` ... Fixed in commit `7633338`. The issue was that `request()` method returns `void`, not `PermissionsBuilder`, so it cannot be...
> @copilot this doesn't seem to work, the location doesn't seems to be accessed, there is no indication in the status bar that the location is being used Fixed in...
> @copilot this still doesn't work, location is not accessed, please implement accessing location properly like on maps apps Fixed in commit `9dcd75f`. The issue was the location updates were...
> @copilot that fails with the following error: > > ``` ... Fixed in commit `469259f`. The `LocationManagerCompat.requestLocationUpdates()` method requires `LocationListenerCompat` instead of `LocationListener`. Updated the `ServiceLocationListener` class to implement...