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

README・vcx-props/project-PlatformToolset.mdで2022対応表記改善(#1952 への追加コミット) #1953

Merged
merged 8 commits into from
Jun 14, 2024

Conversation

tats-u
Copy link
Contributor

@tats-u tats-u commented May 19, 2024

PR対象

  • ドキュメント(md、ヘルプファイル等)

カテゴリ

  • 改善

PR の背景

#1944

2022で普通にビルドできるにもかかわらずREADMEに書いていないのはおかしいです。

なお、zlibライセンスのサクラは特例で全企業Communityを使えるのでVSの新バージョン対応を躊躇する理由はありません。
今後のコミットで2019以前を切り捨ててもいいと思います。

2025以降が出てきた場合の必要な変更を減らしています。
ただし、対応が行われたIssue・PRの欄を増やしています。

あとはメインユーザーであろう企業の社員に対して「使ってばかりいないでたまにはコントリビュートしてくれ、最新版のVSのライセンスの心配はいらないぞ」という遠回しのメッセージを追加しました。

仕様・動作説明

PR の影響範囲

  • README.md
  • vcx-props/project-PlatFormToolset.md

テスト内容

両ファイルを直接・または差分を開いて妥当性確認

関連 issue, PR

参考資料

https://visualstudio.microsoft.com/ja/license-terms/vs2022-ga-community/

@tats-u tats-u marked this pull request as draft May 19, 2024 04:23
@tats-u
Copy link
Contributor Author

tats-u commented May 19, 2024

#1952 に乗っかる形に修正します

@tats-u
Copy link
Contributor Author

tats-u commented May 19, 2024

1952 との差分: berryzplus#34

レビューもそちらへお願いします

#1952berryzplus#34 にマージされたらこちらはクローズします

@tats-u tats-u changed the title READMEで2022対応明記 READMEで2022対応明記(#1952 への追加コミット) May 19, 2024
@tats-u tats-u marked this pull request as ready for review May 19, 2024 05:36
@berryzplus
Copy link
Contributor

あくまで個人的な感想ですが、
バージョン書きたくないです。
(どうせ2025出るし、あっという間に2028出ますから。)

@tats-u tats-u changed the title READMEで2022対応明記(#1952 への追加コミット) README・vcx-props/project-PlatformToolset.mdで2022対応表記改善(#1952 への追加コミット) May 29, 2024
@tats-u
Copy link
Contributor Author

tats-u commented May 29, 2024

@berryzplus 表以外での最新バージョンへの言及は徹底排除しました。2025が出ても表の更新とCommunity板ライセンスへのリンク差し替えだけで済みます。

berryzplus
berryzplus previously approved these changes Jun 3, 2024
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
tats-u and others added 2 commits June 3, 2024 12:39
Co-authored-by: berryzplus <berryzplus@gmail.com>
Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

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

TOC(≒目次?)の変更もお願いします。

@tats-u tats-u requested a review from berryzplus June 7, 2024 10:00
@tats-u
Copy link
Contributor Author

tats-u commented Jun 7, 2024

@berryzplus 修正しました

@tats-u
Copy link
Contributor Author

tats-u commented Jun 7, 2024

コミットのスカッシュが必要ならばお申し付けください

@beru beru added the document ドキュメント label Jun 14, 2024
@beru beru merged commit fd9fcff into sakura-editor:master Jun 14, 2024
14 of 17 checks passed
@tats-u tats-u deleted the readme-2022 branch June 14, 2024 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
document ドキュメント
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants