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

UTF-8で登録されていたpowershellスクリプトを登録し直す #1675

Merged

Conversation

berryzplus
Copy link
Contributor

@berryzplus berryzplus commented May 22, 2021

PR の目的

タイトル通りです。

カテゴリ

  • その他の問題

PR の背景

#1673 (comment) で報告された事象に対処します。

当初、ps1ファイルの文字コードは指定されていませんでした。
 👇
tools/includeGuard/includeGuard.ps1 を追加しました。
 文字コードはUTF-8でした。
 👇
ps1ファイルの文字コードをUTF16LEに変更しました(#1509)
 とくに警告とか出なかったので登録済みファイルの変更は行いませんでした。
 👇
gitのテキストモードを有効にすると、includeGuard.ps1 に差分が出ます。

PR のメリット

  • 差分が出なくなります。

PR のデメリット (トレードオフとかあれば)

  • とくにないと思います。

仕様・動作説明

  • このPRはサクラエディタの仕様・機能を変更しません。

PR の影響範囲

  • git レポジトリのclone時の挙動に影響します。

テスト内容

とくに行っていません。

関連 issue, PR

#1509

参考資料

@sonarcloud
Copy link

sonarcloud bot commented May 22, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@berryzplus berryzplus changed the title UTF-8で登録されていたpowershellスクリプトを登録し直す。 UTF-8で登録されていたpowershellスクリプトを登録し直す May 22, 2021
@AppVeyorBot
Copy link

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

差分が発生したときと同じ手順でチェックアウトし、差分が出ないことを確認しました。

$ mkdir sakura; cd $_
$ git init
$ git config --local core.autocrlf true
$ git config --local core.safecrlf warn
$ git remote add origin https://github.com/sakura-editor/sakura.git
$ git fetch --prune origin pull/1675/head:feature/fix_encoding_of_ps1
$ git checkout feature/fix_encoding_of_ps1
$ git status
On branch feature/fix_encoding_of_ps1
nothing to commit, working tree clean

@berryzplus
Copy link
Contributor Author

レビューありがとうございます。とっととマージしてしまいます。

@berryzplus berryzplus merged commit f5c8024 into sakura-editor:master May 22, 2021
@berryzplus berryzplus deleted the feature/fix_encoding_of_ps1 branch May 22, 2021 12:56
@k-takata
Copy link
Member

ps1ファイルの文字コードをUTF16LEに変更しました。

どのPRなのか、関連PRに書いておいた方がよいと思います。

@berryzplus
Copy link
Contributor Author

ps1ファイルの文字コードをUTF16LEに変更しました。

どのPRなのか、関連PRに書いておいた方がよいと思います。

#1509 でした。

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.

None yet

3 participants