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

Apply firstlinenum to text format #1931

Merged
merged 2 commits into from
Nov 25, 2024
Merged

Conversation

kurank
Copy link
Contributor

@kurank kurank commented Nov 18, 2024

テキストファイルを生成するplaintextbuilderとtopbuilderではfirstlinenumが効いていなかったので、html版を参考に追加してみたのですが、いかがでしょうか。
※ テキスト生成なんて、あまり誰も使っていないのでしょうか...

@kmuto
Copy link
Owner

kmuto commented Nov 23, 2024

@kurank
PRありがとうございます!
修正は良さそうで、rake testのテストを通るようにしていただけますでしょうか。
具体的にはassetsにあるexpected内容のファイルの文字数を変更するだけで通ると思います。

diff --git a/test/assets/syntax_book_index_detail.txt b/test/assets/syntax_book_index_detail.txt
index 000aa099..42c82350 100644
--- a/test/assets/syntax_book_index_detail.txt
+++ b/test/assets/syntax_book_index_detail.txt
@@ -26,11 +26,11 @@
 -----------------------------
    169C     2L   0.2P  第II部 部見出し■□■□■□■□■□■□■□■□■□■□■□■□■□■□■□■□■□■□
 =============================
-  6110C   188L    11P  ch02
+  6111C   188L    11P  ch02
 -----------------------------
     52C     1L   0.0P  第2章 長い章見出し■□■□■□■□■□■□■□■□■□■□■□■□■□■□■□■□■□■□■□■□■□
     10C     1L   0.0P    2.1 ブロック命令
-   784C    27L   1.4P      2.1.1 ソースコード
+   785C    27L   1.4P      2.1.1 ソースコード
    338C     8L   0.5P      2.1.2 図
    709C    22L   1.3P      2.1.3 表
   1154C    56L   2.8P      2.1.4 囲み記事

@kurank
Copy link
Contributor Author

kurank commented Nov 25, 2024

@kmuto ご確認ありがとうございます。文字数更新してまいりました。

@kmuto
Copy link
Owner

kmuto commented Nov 25, 2024

ありがとうございます、テスト通ったのでマージします

@kmuto kmuto merged commit bba805e into kmuto:master Nov 25, 2024
17 checks passed
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.

2 participants