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

「仕様変更」の抽出対応、および分類の日本語化 #13

Merged
merged 5 commits into from
Dec 15, 2018

Conversation

takke
Copy link
Member

@takke takke commented Dec 13, 2018

@takke takke changed the title [ 「仕様変更」の抽出対応、および分類の日本語化 Dec 13, 2018
@KENCHjp
Copy link
Member

KENCHjp commented Dec 13, 2018

生成されたCHANGELOG.md見ましたが、「かなりいい」ですね。
こんなのが自動で出来る世の中ってすばらしい(いや協力してくれるかたが居ての話ですが)

KENCHjp
KENCHjp previously approved these changes Dec 13, 2018
Copy link
Member

@KENCHjp KENCHjp left a comment

Choose a reason for hiding this comment

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

ご協力感謝。

Copy link
Member

@m-tmatma m-tmatma left a comment

Choose a reason for hiding this comment

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

ruby のバージョン変えたのですか?
何か問題ありましたか?

@m-tmatma
Copy link
Member

m-tmatma commented Dec 13, 2018

↑ コミットのコメントに書いてますね。

@takke
Copy link
Member Author

takke commented Dec 13, 2018

改めて、Ruby 2.5 + github_changelog_generator v1.15.0.pre.rc の環境を用意して確認しました。

ちなみに、Ruby 2.5 + github_changelog_generator v1.15.0.pre.beta であれば停止せずに動作します。日本語が通らない問題の検証のため、rc で追加された -require オプションで Ruby のファイルを挟んで内部エンコーディングの確認などをしていたので、今回は Ruby 2.4 + github_changelog_generator v1.15.0.pre.rc にしました。

@m-tmatma
Copy link
Member

ローカルで実行しても同じですか?

@m-tmatma
Copy link
Member

可能であれば、git bisect で問題の commit を特定して、本家にフィードバックしたいです。

今後も使い続けると思うので、使っていく中で
見つけたバグは修正までは、できないかもしれないですが、最低限定、報告はしておきたいです。

@m-tmatma
Copy link
Member

↑ 全部この PR でやらなくても、課題として
issue 登録しておいて、あとで対応するのも
ありだと思ってます。

どれぐらい時間がかかるかと、すぐに対応可能かによります。

@takke
Copy link
Member Author

takke commented Dec 14, 2018

ローカル (Windows10 Pro on VMware) で実行しても同様でした。

2018-12-14_10h07_12

Ctrl+C で見えるスタックトレースが同じなので、github-changelog-generator/github-changelog-generator#668 が本件に相当するようです。

あえて再度報告するまでもないと思います。

github_changelog_generator がアップデートされたときに改めて Ruby 2.5 (以降) を試すということでどうでしょうか。

@m-tmatma
Copy link
Member

ローカルで実行したところ以下のエラーになりました。

db9d666 (vers 2.4 系)

>installChangeLog.bat
>makeChangeLog.bat

INFO: APPVEYOR_REPO_NAME                   =
INFO: APPVEYOR_PULL_REQUEST_HEAD_REPO_NAME =

C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator/parser_file.rb:52:in `gsub': invalid byte sequence in UTF-8 (ArgumentError)
        from C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator/parser_file.rb:52:in `rescue in parse_line!'
        from C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator/parser_file.rb:47:in `parse_line!'
        from C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator/parser_file.rb:34:in `block in parse!'
        from C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator/parser_file.rb:34:in `each'
        from C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator/parser_file.rb:34:in `each_with_index'
        from C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator/parser_file.rb:34:in `parse!'
        from C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator/parser.rb:15:in `parse_options'
        from C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator.rb:28:in `initialize'
        from C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/github_changelog_generator-1.15.0.pre.rc/bin/github_changelog_generator:5:in `new'
        from C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/github_changelog_generator-1.15.0.pre.rc/bin/github_changelog_generator:5:in `<top (required)>'
        from C:/Ruby24-x64/bin/github_changelog_generator:23:in `load'
        from C:/Ruby24-x64/bin/github_changelog_generator:23:in `<main>'
>env-set.bat

>ruby --version
ruby 2.4.5p335 (2018-10-18 revision 65137) [x64-mingw32]

db9d666 を revert したもの (ver 2.5 系)

>installChangeLog.bat
>makeChangeLog.bat


INFO: APPVEYOR_REPO_NAME                   =
INFO: APPVEYOR_PULL_REQUEST_HEAD_REPO_NAME =

Traceback (most recent call last):
        12: from C:/Ruby25-x64/bin/github_changelog_generator:23:in `<main>'
        11: from C:/Ruby25-x64/bin/github_changelog_generator:23:in `load'
        10: from C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/github_changelog_generator-1.15.0.pre.rc/bin/github_changelog_generator:5:in `<top (required)>'
         9: from C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/github_changelog_generator-1.15.0.pre.rc/bin/github_changelog_generator:5:in `new'
         8: from C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator.rb:28:in `initialize'
         7: from C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator/parser.rb:15:in `parse_options'
         6: from C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator/parser_file.rb:34:in `parse!'
         5: from C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator/parser_file.rb:34:in `each_with_index'
         4: from C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator/parser_file.rb:34:in `each'
         3: from C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator/parser_file.rb:34:in `block in parse!'
         2: from C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator/parser_file.rb:47:in `parse_line!'
         1: from C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator/parser_file.rb:52:in `rescue in parse_line!'
C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/github_changelog_generator-1.15.0.pre.rc/lib/github_changelog_generator/parser_file.rb:52:in `gsub': invalid byte sequence in UTF-8 (ArgumentError)
>env-set.bat

>ruby --version
ruby 2.5.3p105 (2018-10-18 revision 65156) [x64-mingw32]

makeChangeLog.bat Outdated Show resolved Hide resolved
installChangeLog.bat Outdated Show resolved Hide resolved
@m-tmatma
Copy link
Member

ローカルで動くのを確認しました。
対応ありがとうございます。

@m-tmatma
Copy link
Member

一箇所日本語のコメントは少し気になりますが。
問題が発生しないのであればいいかなとも思います。

@takke takke merged commit d6f1a12 into sakura-editor:master Dec 15, 2018
@takke
Copy link
Member Author

takke commented Dec 15, 2018

CHANGELOG.mdについて · sakura-editor/sakura Wiki も合わせて修正しておきました。

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.

3 participants