Skip to content

unity-setup@v1.2.3#30

Merged
StephenHodgson merged 1 commit into
mainfrom
fix/editor-lookup
Jul 31, 2025
Merged

unity-setup@v1.2.3#30
StephenHodgson merged 1 commit into
mainfrom
fix/editor-lookup

Conversation

@StephenHodgson

Copy link
Copy Markdown
Member
  • fix editor path lookup

- fix editor path lookup
Copilot AI review requested due to automatic review settings July 31, 2025 03:28
@StephenHodgson StephenHodgson requested a review from a team as a code owner July 31, 2025 03:28

Copilot AI left a comment

Copy link
Copy Markdown

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 fixes the Unity editor path lookup logic by adding an exact version matching preference before falling back to semantic version matching, and bumps the version to 1.2.3.

  • Implements exact version matching as the preferred method for finding Unity editors
  • Maintains semantic version matching as a fallback for compatibility
  • Updates package version to reflect the bug fix

Reviewed Changes

Copilot reviewed 2 out of 5 changed files in this pull request and generated no comments.

File Description
src/unity-hub.ts Adds exact version matching logic before semantic version fallback
package.json Bumps version from 1.2.2 to 1.2.3
Comments suppressed due to low confidence (1)

src/unity-hub.ts:478

  • This closing brace doesn't match the control structure. The else block starting at line 457 should close before this brace, but there's an extra closing brace here that breaks the if-else structure.
        }

@StephenHodgson StephenHodgson merged commit 9a73c76 into main Jul 31, 2025
26 checks passed
@StephenHodgson StephenHodgson deleted the fix/editor-lookup branch July 31, 2025 03:32
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.

2 participants