Skip to content

unity-cli@v1.2.0#20

Merged
StephenHodgson merged 11 commits intomainfrom
development
Oct 6, 2025
Merged

unity-cli@v1.2.0#20
StephenHodgson merged 11 commits intomainfrom
development

Conversation

@StephenHodgson
Copy link
Copy Markdown
Member

  • Refactor LicenseClient.Active with a new ActivateOptions interface for input parameters
  • Refactor Hub installation on windows to invoke UAC when installing hub
  • Added command groups when running help command
  • Added list-project-templates command to list available templates for a given unity editor

- Refactor LicenseClient.Active with a new ActivateOptions interface for input parameters
- Refactor Hub installation on windows to invoke UAC when installing hub
- Added command groups when running help command
- Added list-project-templates command to list available templates for a given unity editor
@StephenHodgson StephenHodgson marked this pull request as ready for review October 6, 2025 13:58
Copilot AI review requested due to automatic review settings October 6, 2025 13:58
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces version 1.2.0 of the unity-cli with several key improvements including better licensing API design, enhanced Windows installation security, and improved CLI organization.

  • Refactored LicenseClient.Active method to use a structured ActivateOptions interface
  • Enhanced Windows installations to invoke UAC for proper administrative privileges
  • Added command grouping for better help organization and a new list-project-templates command

Reviewed Changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/utilities.ts Added -NoProfile flag to PowerShell execution and new EscapeRegex utility function
src/unity-hub.ts Modified Windows installations to use UAC elevation via Start-Process -Verb RunAs
src/unity-editor.ts Refactored template handling with new GetAvailableTemplates method and improved regex escaping
src/logging.ts Renamed CI-specific methods with CI_ prefix for better clarity
src/license-client.ts Introduced ActivateOptions interface to replace multiple parameters in Activate method
src/cli.ts Added command groups, update notifications, and new list-project-templates command
package.json Version bump to 1.2.0 and added update-notifier dependency

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/license-client.ts
Comment thread src/unity-editor.ts Outdated
Comment thread src/cli.ts Outdated
Comment thread src/license-client.ts Fixed
@StephenHodgson StephenHodgson requested a review from Copilot October 6, 2025 14:06

This comment was marked as outdated.

@StephenHodgson StephenHodgson requested a review from Copilot October 6, 2025 15:47
Comment thread src/unity-editor.ts Dismissed
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 12 out of 13 changed files in this pull request and generated 5 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/license-client.ts
Comment thread src/license-client.ts
Comment thread src/unity-editor.ts
Comment thread src/cli.ts
Comment thread src/cli.ts
pull out email validation logic
Comment thread src/license-client.ts Dismissed
@StephenHodgson StephenHodgson merged commit 51a2b48 into main Oct 6, 2025
13 checks passed
@StephenHodgson StephenHodgson deleted the development branch October 6, 2025 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants