Skip to content

fix: replace mojibake with plain ASCII in squad-main-guard.yml#24

Merged
mpaulosky merged 2 commits into
feature/dto-refactor-and-soft-deletefrom
copilot/sub-pr-23
Feb 21, 2026
Merged

fix: replace mojibake with plain ASCII in squad-main-guard.yml#24
mpaulosky merged 2 commits into
feature/dto-refactor-and-soft-deletefrom
copilot/sub-pr-23

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 20, 2026

Garbled mojibake strings (from a bad UTF-8 → Windows-1252 encoding conversion) in the guard workflow's script block made CI log output and error messages unreadable.

Changes

  • .github/workflows/squad-main-guard.yml — replaced all corrupted character sequences with plain ASCII equivalents:
Mojibake Replacement
ΓÇö (em dash) -
✅ (checkmark) [OK]
🚫 (warning emoji) [WARNING]
⚠️ (warning emoji) [NOTE]

All occurrences were in inline comments and user-facing core.info/core.setFailed strings between lines 72–127. No logic was changed.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

….yml

Co-authored-by: mpaulosky <60372079+mpaulosky@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP to address feedback on DTOs and soft-delete audit trail fix: replace mojibake with plain ASCII in squad-main-guard.yml Feb 20, 2026
Copilot AI requested a review from mpaulosky February 20, 2026 23:56
@mpaulosky mpaulosky marked this pull request as ready for review February 21, 2026 00:08
Copilot AI review requested due to automatic review settings February 21, 2026 00:08
Copy link
Copy Markdown
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 fixes encoding issues (mojibake) in the squad-main-guard workflow file that made CI logs and error messages unreadable. The corrupted UTF-8 character sequences resulted from a bad encoding conversion and have been replaced with plain ASCII equivalents.

Changes:

  • Replaced all mojibake character sequences with plain ASCII text in comments and user-facing messages
  • No logic or functionality changes—only text cleanup for improved readability

@github-actions
Copy link
Copy Markdown

Test Results Summary

70 tests   70 ✅  2s ⏱️
 5 suites   0 💤
 5 files     0 ❌

Results for commit a495f2b.

@mpaulosky mpaulosky merged commit 633e702 into feature/dto-refactor-and-soft-delete Feb 21, 2026
13 checks passed
@mpaulosky mpaulosky deleted the copilot/sub-pr-23 branch February 21, 2026 00:12
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (feature/dto-refactor-and-soft-delete@707395d). Learn more about missing BASE report.

Additional details and impacted files
@@                           Coverage Diff                           @@
##             feature/dto-refactor-and-soft-delete      #24   +/-   ##
=======================================================================
  Coverage                                        ?   31.82%           
=======================================================================
  Files                                           ?       28           
  Lines                                           ?      597           
  Branches                                        ?       32           
=======================================================================
  Hits                                            ?      190           
  Misses                                          ?      388           
  Partials                                        ?       19           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

3 participants