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

copyright を 2018 に変更 #232

Merged
merged 1 commit into from
Jul 8, 2018

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented Jul 8, 2018

copyright を 2018 に変更

@m-tmatma m-tmatma added the license ライセンス関連 label Jul 8, 2018
@m-tmatma m-tmatma added this to the next release milestone Jul 8, 2018
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.

LGTMです。

@@ -34,7 +34,7 @@ LANGUAGE LANG_ENGLISH, SUBLANG_DEFAULT
#pragma code_page(932)
#endif //_WIN32

#define S_COPYRIGHT "Copyright (C) 2011-2017 by Lucien & Collaborators"
#define S_COPYRIGHT "Copyright (C) 2011-2018 by Lucien & Collaborators"
Copy link
Contributor

Choose a reason for hiding this comment

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

PR内容とは関係ないですが・・・。
いつかどこかで話さないといけないトピック。

翻訳じゃなくて著作者変わっとるやんけ、という話。

Copy link
Member

Choose a reason for hiding this comment

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

翻訳の場合、普通はどういうふうに書くのが一般的なんですかね

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

今内容を把握。dll のほうの著作者かと思ってましたが、よく確認してみたらサクラエディタ本体ダイアログの Copyright が変わっちゃう所でしたね。たしかにこれは問題。(とはいえやはりどのような書き方が好ましいのか分かってない)

Copy link
Member

Choose a reason for hiding this comment

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

http://d.hatena.ne.jp/sardine/20060912

Copyright of the original © 2004-2006 W3C® (MIT, ERCIM, Keio), All Rights Reserved. Translated by [ここに翻訳者名]. W3C liability, trademark and document use rules apply for the original and the translation."

みたいな?

Copy link
Member

Choose a reason for hiding this comment

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

この話は #233 で続きやりましょうかね~

@kobake
Copy link
Member

kobake commented Jul 8, 2018

バージョンダイアログの表示が 2018 に変わっていることを確認。

@kobake
Copy link
Member

kobake commented Jul 8, 2018

exe, dll のプロパティ側の表示も 2018 に変わったことを確認。

@kobake
Copy link
Member

kobake commented Jul 8, 2018

@m-tmatma さん、およびみなさん
PR 作成時には PR 本文で「どの範囲が変更されるのか」明示いただけると助かります。

自分は最初 exe, dll のプロパティのみ変更されるものと思い込んでしまっていましたが、よくよく見たらバージョンダイアログ表示も変わっていることに後から気づきました。

Copy link
Member

@kobake kobake left a comment

Choose a reason for hiding this comment

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

著作者の件はさておき、この PR の変更内容については LGTM です。

@kobake kobake merged commit f0acb45 into sakura-editor:master Jul 8, 2018
@m-tmatma
Copy link
Member Author

m-tmatma commented Jul 8, 2018

PR 作成時には PR 本文で「どの範囲が変更されるのか」明示いただけると助かります。

リソース DLL しか変更されないつもりでした。
バージョン表示も変わるのは想定しませんでした。

@m-tmatma
Copy link
Member Author

m-tmatma commented Jul 8, 2018

著作者の件はさておき、

#233 を登録しました。

@m-tmatma m-tmatma deleted the feature/copyright-2018 branch July 8, 2018 02:11
@ds14050 ds14050 added the license ライセンス関連 label Sep 18, 2018
@berryzplus
Copy link
Contributor

あらためまして、新年あけましておめでとうございます 😄

HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
license ライセンス関連
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants