Skip to content

fix: replace garbled mojibake characters with clean ASCII in comments#29

Open
57388sp wants to merge 2 commits into
mainfrom
fix/garbled-text-in-comments
Open

fix: replace garbled mojibake characters with clean ASCII in comments#29
57388sp wants to merge 2 commits into
mainfrom
fix/garbled-text-in-comments

Conversation

@57388sp
Copy link
Copy Markdown
Collaborator

@57388sp 57388sp commented May 11, 2026

Several Python source files contained mojibake characters — Windows CP1252
bytes misread as UTF-8 — in docstrings and comments. This was likely
introduced when files were edited in a non-UTF-8 editor.

Changes:

  • Replace ΓÇö with - (em-dash mangled)
  • Replace ΓǪ with ... (ellipsis mangled)
  • Replace ΓåÆ with -> (arrow mangled)
  • Replace Γ£ô with [OK] (checkmark mangled)
  • Remove UTF-8 BOM from cluster_info.py
  • Blank hardcoded internal IPs, passwords, and lab aggregate name
    (AGGR_NAME, ONTAP_PASS, CLUSTER_PASS, node IPs) — these should
    never be committed; users set them via env vars or --env-file

No logic changes. Tests: 154/154 pass. Ruff: clean.

@github-actions
Copy link
Copy Markdown

Test Report missing or unfilled.

This PR touches python/, ansible/, or terraform/, so a populated Test Report section is required in the PR body. See TESTING.md for what to capture (environment, ONTAP version, first-run output, idempotency check, teardown).

This is a soft gate - your CI checks are unaffected - but reviewers will not approve until the report is filled in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants