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.

Sign up for GitHub

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

Jump to bottom

v2.4.2リリース準備 #1872

Merged
merged 1,389 commits into from
Dec 3, 2022
Merged

v2.4.2リリース準備 #1872

merged 1,389 commits into from
Dec 3, 2022

Conversation

berryzplus
Copy link
Contributor

PR の目的

v2.4.2リリースのため

カテゴリ

  • リリース準備作業

PR の背景

v2.4.2リリースの準備のために変更の取り込みを行います。

PR のメリット

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

仕様・動作説明

v2.4.2ブランチ(内容はv2.4.1)に最新のmasterの変更をマージします。

PR の影響範囲

テスト内容

テスト1

手順

関連 issue, PR

参考資料

berryzplus and others added 30 commits October 12, 2021 00:20
@berryzplus
Merge pull request sakura-editor#1738 from berryzplus/feature/refacto…
a671199 
…ring_version_info

バージョン情報をリファクタリングする
@berryzplus
累積コミット数の取得にgit rev-listを使う
135b33b
@berryzplus
Merge pull request sakura-editor#1743 from berryzplus/feature/use_git…
1a8ce4c 
…-rev-list_to_count_commits

累積コミット数の取得にgit rev-listを使う
@sanomari
Merge pull request sakura-editor#1742 from sanomari/feature/use_windo…
7858990 
…wsx_h

Windows 10 SDKのメッセージクラッカーを使えるようにする
@sanomari
グローバル関数GetEditWndを追加
3ca63d7
@sanomari
冗長なメンバ変数 CEditVIew::m_pcEditWnd を削除する
67f8ac8 
CEditWndのインスタンスにはグローバル変数のようにアクセスできる。このため、コンストラクタでCEditWndのインスタンスを要求してクラス内外でのアクセスに流用する構造は無駄と考えられる。
@sanomari
CEditVIew::m_pcEditWnd置換1 CEditView*.cpp内97箇所
d83ad79
@sanomari
CEditVIew::m_pcEditWnd置換2 パターン置換49箇所
2e458ad 
置換パターン
\b((m_)?pc?)?(Edit)?View(Dst)?->m_pcEditWnd->
@sanomari
CEditVIew::m_pcEditWnd置換3 手作業置換7箇所
37b1cb2
@sanomari
冗長なメンバ変数 CEditDoc::m_pcEditWnd を削除する
e64af20
@sanomari
CEditDoc::m_pcEditWnd置換1 CEditDoc内46箇所
000c066
@sanomari
CEditDoc::m_pcEditWnd置換2 パターン置換66箇所
2bd655f 
\b((m_)?pc?)?(Edit)?Doc(Ref)?->m_pcEditWnd->
@sanomari
CEditDoc::m_pcEditWnd置換3 手作業置換
889939f
@sanomari
冗長なメンバ変数 CSplitterWnd::m_pcEditWnd削除 を削除する
f477971
@sanomari
グローバル変数g_pcEditWndを削除する
4b341c4 
CEditWndのインスタンスはクラスのstatic変数に保持されるので、グローバル変数に持つと二重保持になり無駄です。
@sanomari
Merge pull request sakura-editor#1744 from sanomari/feature/fix_refer…
3b1a659 
…ence_of_CEditWnd

編集ウインドウへの参照をリファクタリングする
@sanomari
GetEditWndの指摘修正
22edbce 
変数宣言のconstを削除し、c style-castしなくて良いようにする
@sanomari
CMiniMapView追加
c83e5e5
@sanomari
Merge pull request sakura-editor#1747 from sanomari/feature/fix_GetEd…
8565cdb 
…itWnd

GetEditWndの指摘修正
@sanomari
Merge pull request sakura-editor#1748 from sanomari/feature/add_cmini…
0dd4391 
…mapview

ミニマップの実装クラスを分割する
ダイアログの位置とサイズを記憶する処理を補完候補ダイアログに限って復元する
8212d55
似通った変数名を変更
62d06ad
リストボックスからのテキスト取得処理で取りこぼしが生じているのを修正
c899648
ApiWrap::List_GetText()のテストを追加
978a9a5
タイプミスを修正
7508075
@beru
トリプルクリック時に次の行の先頭にURLがあるとそれが選択されてしまう現象が起きないように対策
b18d540
@beru
カーソル前を削除する処理 CViewCommander::Command_DELETE_BACK で CDocEditor::m_nOpe…
5384616 
…BlkRedawCount をインクリメントする処理を追加

キャレット移動の CMoveCaretOpe 追加では再描画は必要無いはず
補完候補一覧ダイアログの不具合修正 (sakura-editor#1750) from kazasaku/feature/fix_word…
42568b8 
…_completion
.vsディレクトリを除外指定に加える
8a043b3
ChmSourceConverterのターゲットを「.NET Framework 4.7.2」に変更
dd7d091
GermanAizek and others added 13 commits July 18, 2022 17:59
@GermanAizek
Code modernize C++17, refactor and minor optimize
c491dd4
@berryzplus @GermanAizek
Additional fixes review by berryzplus <berryzplus@gmail.com>
1fc1801 
Signed-off-by: lainon <GermanAizek@yandex.ru>
@berryzplus
using namespace std::string_literals
1c71b64
@berryzplus
Fix the maximum length of path strings.
c48328f
@berryzplus
指摘修正
d20e9d0 
Shiftを押さなかった場合にスクロールしなくなってしまっていたのを修正。
@berryzplus
Merge pull request sakura-editor#1860 from GermanAizek/master
eca9316 
Code modernize C++17, refactor and minor optimize
@eltociear
CMakeLists.txtのコメントタイポ修正
156fc3c 
libary -> library
@beru
Merge pull request sakura-editor#1862 from eltociear/patch-1
91a09d7 
CMakeLists.txtのコメントタイポ修正
@berryzplus
テストケース修正
b76c2e0 
マルチユーザー設定ファイル(sakura.exe.ini)のテストが失敗したときに、テストのために作成したexe.iniが削除されず、他ケースが失敗していたのを修正
@berryzplus
マルチユーザー設定ファイルの不具合修正
128f442 
マルチユーザー設定ファイルのUserRootFolderに
2:ドキュメントフォルダ、3:デスクトップを指定した場合、
OneDriveのフォルダが取得されてしまう不具合を修正する。

この不具合はOneDriveをセットアップしないと再現しないのでCI/CDではチェックできない。
@berryzplus
Merge pull request sakura-editor#1864 from berryzplus/feature/fix_tes…
64fb6f7 
…t_for_exeini

マルチユーザー設定ファイル(sakura.exe.ini)のテストケース修正
@berryzplus
Merge pull request sakura-editor#1865 from berryzplus/feature/fix_exe…
b0b7b83 
…ini_settings

マルチユーザー設定ファイルの不具合修正
@berryzplus
Merge branch 'main' into release/v2.4.2
5170a70
@berryzplus berryzplus mentioned this pull request Nov 30, 2022
Closed
9 tasks
@AppVeyorBot
Copy link

✅ Build sakura 1.0.4195 completed (commit 26ff9d4dc0 by @berryzplus)

@berryzplus
Copy link
Contributor Author

マージミスがあったようなので対応できるまでdraftにします。

@berryzplus berryzplus marked this pull request as draft December 1, 2022 01:07
@AppVeyorBot
Copy link

✅ Build sakura 1.0.4196 completed (commit 145cc9e68c by @berryzplus)

@berryzplus
Copy link
Contributor Author

SonarCloudのサマリです。
https://sonarcloud.io/summary/new_code?id=berryzplus_sakura-editor&branch=feature%2Fcurrent_master

@berryzplus berryzplus marked this pull request as ready for review December 1, 2022 02:42
@berryzplus berryzplus mentioned this pull request Dec 2, 2022
Closed
@beru beru added IMPORTANT 早急に解消すべきもの Release Release作業チケット【ChangeLog除外】 labels Dec 2, 2022
beru
beru approved these changes Dec 2, 2022
View reviewed changes
Copy link
Contributor

@beru beru left a comment

Choose a reason for hiding this comment

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

x64版のインストーラーをダウンロードしてインストール後に起動して簡単な操作を行いました。

あまり動作確認をしっかりしていませんがリリースして問題無いと思います。
もし問題が見つかったら後で v2.4.2.1 をリリースしてカバーすれば良いかなと…。

@berryzplus
Copy link
Contributor Author

レビューありがとうございます。マージして #1870 を進めます。

@berryzplus berryzplus merged commit ff56179 into sakura-editor:release/v2.4.2 Dec 3, 2022
@berryzplus berryzplus deleted the feature/current_master branch December 3, 2022 11:01
@tats-u tats-u mentioned this pull request May 19, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@m-tmatma m-tmatma m-tmatma approved these changes

@beru beru beru approved these changes

Assignees
No one assigned
Labels
IMPORTANT 早急に解消すべきもの Release Release作業チケット【ChangeLog除外】
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.