Skip to content

Conversation

StephenHodgson
Copy link
Member

  • add support for industry license types
  • post step no longer fails when license cannot be returned

- add support for industry license types
- post step no longer fails when license cannot be returned
@StephenHodgson StephenHodgson marked this pull request as ready for review August 5, 2025 19:45
@Copilot Copilot AI review requested due to automatic review settings August 5, 2025 19:45
@StephenHodgson StephenHodgson requested a review from a team as a code owner August 5, 2025 19:45
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

This PR adds support for Unity industry license types and improves the post-step error handling to prevent failures when licenses cannot be returned.

  • Add new industry license type to the supported license types enum
  • Update license activation logic to handle industry licenses alongside existing personal, professional, and floating licenses
  • Change post-step behavior to log errors instead of failing when license deactivation fails

Reviewed Changes

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

Show a summary per file
File Description
src/types.ts Introduces LicenseType enum with all supported license types including new industry type
src/licensing-client.ts Updates function signatures to use LicenseType enum and adds industry license activation logic
src/deactivate.ts Changes post-step to log errors instead of failing when license return fails
src/activate.ts Updates license validation and activation to support industry licenses using the enum
package.json Version bump to 1.1.0 and dependency updates
action.yml Updates documentation to reflect new industry license support and lowercase naming
README.md Updates documentation with new license types and corrected examples

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.

1 participant