Skip to content

Conversation

@StephenHodgson
Copy link
Member

@StephenHodgson StephenHodgson commented Nov 29, 2025

  • prevent utp table from rendering in ci builds

- fix utp table rendering in ci builds
Copy link

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 bumps the version to 1.7.2 and prevents UTP (Unity Test Platform) table rendering in CI builds.

Key changes:

  • Extracted UTP processing logic into a separate printUTP function for better code organization
  • Changed UTP telemetry log file extension from .json to .json.log and removed pretty-printing
  • Modified stdout writing to use direct process.stdout.write() for regular log lines, avoiding unnecessary table manipulation

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/unity-logging.ts Refactored UTP handling by extracting printUTP function, changed telemetry log format, and modified stdout writing behavior
src/cli.ts Added default value documentation to log-level option
package.json Version bump to 1.7.2
package-lock.json Version bump to 1.7.2

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

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 3 out of 4 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@StephenHodgson StephenHodgson merged commit 02687a3 into main Nov 29, 2025
2 checks passed
@StephenHodgson StephenHodgson deleted the dev/logging branch November 29, 2025 21:18
StephenHodgson added a commit that referenced this pull request Nov 29, 2025
- prevent utp table from rendering in ci builds
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