Copilot
Copilot
> @copilot run the ruff linter (through `invoke` tasks) and fix all linting issues Fixed all ruff linting issues. Removed 3 unused variables from test file (commit 0a2be4d).
> @copilot rebase on top of current `main` head Rebased on top of current main head (43cb6b6). Resolved merge conflict in cone.py where the new scale() method from main was...
> @copilot please also do this for `PlanarSurface` Added transform() implementation to PlanarSurface with scale component handling and comprehensive tests (commit fd3dd45). > if we are re-adding these implementations, we...
> Instead of adding a new property for --api-levels , can we make --api-version work for the same propose, adding one or several ? Done! The `--api-version` argument now supports...
> @copilot This PR may be too narrowly focused. `null` should work to unset all of the other options at the CLI too. The implementation already works for all options,...
> @copilot you did not run the formatter as instructed Fixed in commit 81d4e0d. Ran `npx hereby format` to correct indentation.
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.
@mdewey I've opened a new pull request, #25425, to work on those changes. Once the pull request is ready, I'll request review from you.
@ikesau I've opened a new pull request, #5705, to work on those changes. Once the pull request is ready, I'll request review from you.