Copilot
Copilot
@teolemon š 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 changes looks good. Please try writing minimal tests Added minimal tests in commit cac5f4d: - `test_always_kill_tempfile_cleanup`: Verifies the monitor properly cleans up temporary files after execution - `test_parse_remote_data_tempfile_cleanup`:...
> @copilot fix linting issues: > ``` > Running SwiftFormat... ... Fixed in 48d845a - removed all trailing spaces and corrected extension access control placement.
> @copilot fix tests: > > ``` ... Fixed in 5efcd25 - added AppZone source files to Xcode project build targets (Shared-iOS, Shared-watchOS, Tests-Shared, and Tests-App). The "cannot find type...
@aramase š 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 > In build/test-aot.ps1, we'd need to change the $expectedWarningCount to 52 with this PR. Updated in commit fbd42ae.
> @copilot Iām not a fan of treating audio and video differently. Since it's a launching option why not use the [ApplicationView.PreferredLaunchWindowingMode](https://learn.microsoft.com/en-us/uwp/api... Good suggestion! I've implemented the PreferredLaunchWindowingMode approach and...
> @copilot After some testing I'm not sure this is what we want. It works, but it also forces the app into fullscreen mode even when opening the app normally....
> @copilot There is a `WindowService` that handles window sizing operations. Setting `PreferredLaunchWindowingMode` in that service would make more sense. There has also been a conflict with the main br......
> @copilot You did not resolve the conflict properly, so I resolved the merge conflicts for you. Please move the full-screen logic to the WindowService again. Done! The fullscreen logic...