Skip to content
This repository was archived by the owner on Nov 13, 2025. It is now read-only.

chore: remove bom encode config#42

Merged
ArgoZhang merged 1 commit into
masterfrom
chore-editor
Nov 9, 2025
Merged

chore: remove bom encode config#42
ArgoZhang merged 1 commit into
masterfrom
chore-editor

Conversation

@ArgoZhang

Copy link
Copy Markdown
Member

Link issues

fixes #41

Summary By Copilot

Regression?

  • Yes
  • No

Risk

  • High
  • Medium
  • Low

Verification

  • Manual (required)
  • Automated

Packaging changes reviewed?

  • Yes
  • No
  • N/A

☑️ Self Check before Merge

⚠️ Please check all items below before review. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • Merge the latest code from the main branch

Copilot AI review requested due to automatic review settings November 9, 2025 07:25
@ArgoZhang ArgoZhang merged commit 7bb8f5a into master Nov 9, 2025
5 checks passed
@ArgoZhang ArgoZhang deleted the chore-editor branch November 9, 2025 07:25

Copilot AI left a comment

Copy link
Copy Markdown

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 removes the UTF-8 BOM charset configuration override for specific file types in the .editorconfig file. The change simplifies the configuration by removing an override that set charset = utf-8-bom for CS, CSS, JS, JSON, HTML, Razor, TXT, and LOG files, allowing these files to use the global charset = utf-8 setting instead (which is defined on line 11 for all files).

Key Changes:

  • Removed UTF-8 BOM charset override for specific file types, standardizing on UTF-8 without BOM across the codebase

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

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore: remove bom encode config

2 participants