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

.gitignoreにビルド構成の変更を反映する #1676

Merged
merged 1 commit into from
May 22, 2021

Conversation

berryzplus
Copy link
Contributor

@berryzplus berryzplus commented May 22, 2021

PR の目的

タイトル通りです。

カテゴリ

  • その他の問題

PR の背景

#1673 (comment) で報告された問題の対処です。

#1670 でビルド構成を変更しました。
 その際、 .gitignore を変更しませんでした。
 👇
普通にgit cloneした状態のworking treeをビルドすると、中間生成物が変更ファイルにリストされます。

PR のメリット

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

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

仕様・動作説明

PR の影響範囲

テスト内容

新規にgit cloneしたディレクトリでVisual Studioでビルドし、中間生成物がリストされなくなったのを確認しました。

関連 issue, PR

#1670
#1673

参考資料

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.

VS上でのビルド・ローカルでのバッチビルドともに差分は発生しませんでした。
問題ないと思います。

@berryzplus
Copy link
Contributor Author

レビューありがとうございます。これも実害あるのでとっととマージしちゃいます。

@berryzplus berryzplus merged commit 0f97078 into sakura-editor:master May 22, 2021
@berryzplus berryzplus deleted the feature/fix_ignorelist branch May 22, 2021 13:06
This pull request was closed.
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.

1 participant