Skip to content

Conversation

@MariusStorhaug
Copy link
Member

Description

This pull request makes a minor change to the Planets.ps1 files in both test repositories. The only update is the addition of a Unicode Byte Order Mark (BOM) at the beginning of each file to ensure proper encoding.

  • Added a Unicode BOM to the start of Planets.ps1 in both srcTestRepo and srcWithManifestTestRepo to enforce UTF-8 encoding.

@MariusStorhaug MariusStorhaug self-assigned this Oct 12, 2025
@MariusStorhaug MariusStorhaug requested a review from a team as a code owner October 12, 2025 11:44
Copilot AI review requested due to automatic review settings October 12, 2025 11:44
Copy link
Contributor

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 UTF-8 Byte Order Mark (BOM) encoding to PowerShell test files to ensure proper character encoding. The change involves adding a Unicode BOM character at the beginning of two Planets.ps1 files.

  • Added UTF-8 BOM to PowerShell files for consistent encoding
  • Updated test repository files to include proper encoding headers

Reviewed Changes

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

File Description
tests/srcWithManifestTestRepo/src/variables/public/Planets.ps1 Added UTF-8 BOM to enforce proper encoding
tests/srcTestRepo/src/variables/public/Planets.ps1 Added UTF-8 BOM to enforce proper encoding

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

@MariusStorhaug MariusStorhaug merged commit f84654d into main Oct 12, 2025
19 of 22 checks passed
@MariusStorhaug MariusStorhaug deleted the UTF8BOM branch October 12, 2025 11:55
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in PSModule Framework Oct 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants