Skip to content

Conversation

StephenHodgson
Copy link
Member

@StephenHodgson StephenHodgson commented Sep 27, 2025

  • refactored to use new unity-cli
  • added additional auto-update-hub flag to enable/disable unity hub updates

- refactored to use new unity-cli
@StephenHodgson StephenHodgson marked this pull request as ready for review September 27, 2025 16:00
@Copilot Copilot AI review requested due to automatic review settings September 27, 2025 16:00
Copilot

This comment was marked as outdated.

Copy link
Contributor

@Copilot 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 19 out of 31 changed files in this pull request and generated no new comments.


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

@brenocogu
Copy link

Hey Stephen! This seems to tackle a problem I'm having regarding pipelines on MacOS
Looking forward to this release 🎉

@StephenHodgson
Copy link
Member Author

StephenHodgson commented Sep 30, 2025

Hey Stephen! This seems to tackle a problem I'm having regarding pipelines on MacOS Looking forward to this release 🎉

@brenocogu you should be able to use v1 MacOS already.

This refactor just integrates my new Unity-CLI tool

@brenocogu
Copy link

@StephenHodgson yeah I've got a problem with a runner that didn't had homebrew + wget installed
The install-unityhub-macos.sh contains linux-only code (wget) and it sent my pipeline to heavens
Gladly it was easily solved - just switching to a curl on my fork repo solved it (didn't wanted to have environmental changes on runner). I was planning to open a PR but it seems not to be the case anymore as this release attacks this very issue.
In any situation, if I can be of any help, let me know ;)
Cudos and thanks again!

@StephenHodgson
Copy link
Member Author

@brenocogu was this a self hosted runner?

You would need to make sure your environment is set up with the default installations from the GitHub runner images repo.

@brenocogu
Copy link

brenocogu commented Sep 30, 2025

@StephenHodgson Yes it was a self-hosted, also it was easy to fix it -> brenocogu@b661a57
I was just about to run some tests using other runners but I'll hold it off to validate the v2.
I'm keen to test-out the v2 of this action when it releases and help it work seamlessly on any runner OS.

@StephenHodgson StephenHodgson merged commit ce42954 into main Oct 6, 2025
49 checks passed
@StephenHodgson StephenHodgson deleted the development branch October 6, 2025 12:12
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