Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Copilot chat: fix typescript encoding format #654

Conversation

TaoChenOSU
Copy link
Contributor

Motivation and Context

Two typescript files are saved as UTF-8 with BOM, which will cause eslint to generate warnings.

Description

Resave the two files to UTF-8 format.

Contribution Checklist

@TaoChenOSU TaoChenOSU added PR: ready for review All feedback addressed, ready for reviews samples labels Apr 25, 2023
@adrianwyatt adrianwyatt enabled auto-merge (squash) April 26, 2023 00:11
Copy link
Member

@Vivihung Vivihung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the fix!

@adrianwyatt adrianwyatt merged commit 08956e4 into microsoft:main Apr 26, 2023
11 checks passed
dluc pushed a commit that referenced this pull request Apr 29, 2023
### Motivation and Context
Two typescript files are saved as UTF-8 with BOM, which will cause
eslint to generate warnings.

### Description
Resave the two files to UTF-8 format.
dehoward pushed a commit to lemillermicrosoft/semantic-kernel that referenced this pull request Jun 1, 2023
### Motivation and Context
Two typescript files are saved as UTF-8 with BOM, which will cause
eslint to generate warnings.

### Description
Resave the two files to UTF-8 format.
golden-aries pushed a commit to golden-aries/semantic-kernel that referenced this pull request Oct 10, 2023
### Motivation and Context
Two typescript files are saved as UTF-8 with BOM, which will cause
eslint to generate warnings.

### Description
Resave the two files to UTF-8 format.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: ready for review All feedback addressed, ready for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants