Copilot

Results 4443 comments of Copilot

@jonathanpeppers Unfortunately I hit an unexpected error while trying to work on issue #192. I've automatically reported this to GitHub. You can ask me to try again later by unassigning...

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

> @copilot Add an injectTestingRouterStore function so consumers can avoid casting. Added the `injectTestingRouterStore()` function as requested. This eliminates the need for manual casting and provides direct access to TestingRouterStore...

> @copilot The inject function should support InjectOptions similar to TestBed.inject. It should also be possible to pass a component and use its Injector to get a local router store....

> @copilot from the comments on the issue this PR is linked to, the quote behavior was only needed on windows. The non-windows targets did not have this problem. Updated...

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

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

> @copilot Could you refactor the code to have only one getBoolEnv() function in a centralized package? Futhermore, could the environment variable names be refactored as central constants? Done! I've...

> @copilot could you extend the use of `GetBoolEnv()` to all packages of the provider and also extract any additional environment variable names to constants? Done! I've extended `GetBoolEnv()` to...

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