Copilot

Results 4904 comments of Copilot

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. > [!WARNING] > > > Firewall rules blocked me...

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. > [!WARNING] > > > Firewall rules blocked me...

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

> @copilot Address the PR feedback at https://github.com/microsoft/FluidFramework/pull/25907#pullrequestreview-3501417038 I don't have access to external GitHub URLs. Could you please paste the specific feedback from that review here, or let me...

> @copilot just keep to string or string[] Done! I've simplified the implementation to use `string | string[]` as requested. Removed the complex `TokenColorValue` objects and now multiple colors are...

> @copilot fix tests Fixed the tests by adding comprehensive GTK mocking in conftest.py and updating the journal test to work without GTK dependencies. All tests now pass (94 passed,...

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

> @copilot the evaluation of > > `!"off".equalsIgnoreCase(System.getProperty(BEEP_ENABLED))`... Done in commit e9c2a1d. Changed BEEP_ENABLED to a public static boolean field initialized with the property evaluation. Also removed the test cases...

The agent encountered an error and was unable to start working on this issue: job builder encountered known error and finalized job: job cannot be created due to repository rules...

> @copilot please resolve the comment. > Please run `/eng/scripts/Export-API.ps1 provisioning` to refresh the API, and update changelog file. Completed: - Ran `eng/scripts/Export-API.ps1 provisioning` to refresh API listings - Updated...