Copilot
Copilot
> @copilot, please reference the code review comments above and update the PR. All PR review comments have been addressed in commits 3af00e3 and 3264d4c: - Removed weight for alphabetical...
> You didn't update the scon properly. > Take example of this : > https://github.com/godotengine/godot-cpp/blob/master/test/SConstruct ... Updated the SConstruct to follow the godot-cpp pattern. The documentation is now embedded directly...
> Check this page to make sure you did thing properly: > https://docs.godotengine.org/en/latest/tutorials/scripting/cpp/gdextension_docs_system.html > ... Verified the SConstruct implementation matches the godot-cpp pattern exactly. Also replaced the ReadTheDocs API reference...
> `FmodServer.get_singleton()` is not necessary. FmodServer is a declared a a singleton so it can be used from gdscript simply using `FmodServer.some_method` > @copilot Updated all API documentation examples to...
> @copilot Lets remove the changes made in build.gradle from the PR. Done. Reverted msal/build.gradle to match dev branch (commonVersion and common4jVersion now "1.0.+"). Commit: 126f213
> @copilot could you also not include all the common changes in the PR? Done. Reverted common submodule to dev branch state (8bdb7ba), removed Version 8.1.1 from changelog, and reverted...
> @copilot .turbo should be git ignored It is! Added in commit 6e852f7. The .turbo/ directory is on line 20 of .gitignore.
> @copilot fix lint errors Fixed all lint errors in commit 10f8742: **Changes made:** - Replaced `assert()` with `assert.ok()` throughout test files (46 fixes) - Removed console statements or added...
> @copilot please review failures i.e. https://helixr1107v0xdeko0k025g8.blob.core.windows.net/dotnet-sdk-refs-pull-52112-merge-4c6167923cd5459290/Microsoft.NET.Build.Tests.dll.9/3/console.e907141d.log?h... The log URL appears to be expired or inaccessible. I've verified: - All ApiCompatibility tests pass (260 tests) - The build succeeds without...
> @Copilot the build failed Fixed using statement ordering to comply with .editorconfig (System.* namespaces first, then alphabetically sorted). See commit cd40fc7. > [!WARNING] > > > Firewall rules blocked...