Skip to content

Conversation

@pogin503
Copy link
Member

@pogin503 pogin503 commented Sep 8, 2014

atom.xml中の警告を修正

Build Warning: Layout 'nil' requested in atom.xml does not exist.
この警告が出るので nil -> nullに変更

Jekyll2.0から非推奨な設定の書き換え

Warning:
Deprecation: The 'pygments' configuration option has been renamed to 'highlighter'. Please update your config file accordingly. The allowed values are 'rouge', 'pygments' or null.

@see mmistakes/jekyll-theme-hpstr#25

Build Warning: Layout 'nil' requested in atom.xml does not exist.
この警告が出るので nil -> nullに変更
Warning:
     Deprecation: The 'pygments' configuration option has been renamed to 'highlighter'. Please update your config file accordingly. The allowed values are 'rouge', 'pygments' or null.

@see mmistakes/jekyll-theme-hpstr#25
syohex added a commit that referenced this pull request Sep 8, 2014
@syohex syohex merged commit b156c7e into emacs-jp:master Sep 8, 2014
@syohex
Copy link
Member

syohex commented Sep 8, 2014

Thanks

@pogin503
Copy link
Member Author

pogin503 commented Sep 8, 2014

お手数をお掛けします。マージありがとうございます。

こういう小さい変更でもpull requestとしてやっているのですが、自分でpushしてもいいんでしょうか? 大きい変更ならpull requestとして送った方がいいと思うのですが。

@syohex
Copy link
Member

syohex commented Sep 8, 2014

自分でpushしてもいいんでしょうか?

はい, 構いません.

大きい変更というより, 誰かに聞きたい, 意見を求めたいという場合は PRしていただいた
方がいいかと思います. しかし PRは遅くなりがちなので, ガンガン pushしてもらった
方が私としては助かります. ライブラリ等と違い古いものを誰かが抱え込むなんてことは
ないので, どんどん追加, 修正していけばよいと思います.

@supermomonga
Copy link
Contributor

大きい変更というより, 誰かに聞きたい, 意見を求めたいという場合は PRしていただいた
方がいいかと思います. しかし PRは遅くなりがちなので, ガンガン pushしてもらった
方が私としては助かります. ライブラリ等と違い古いものを誰かが抱え込むなんてことは
ないので, どんどん追加, 修正していけばよいと思います.

そういう方針なのですね。
これを記載したCONTRIBUTIONファイルを置いておけば他のCollaboratorや新しくCollaboratorになった方にも伝わって良いかもしれません。
https://github.com/blog/1184-contributing-guidelines

@pogin503
Copy link
Member Author

pogin503 commented Sep 8, 2014

大きい変更というより, 誰かに聞きたい, 意見を求めたいという場合は PRしていただいた
方がいいかと思います. しかし PRは遅くなりがちなので, ガンガン pushしてもらった
方が私としては助かります. ライブラリ等と違い古いものを誰かが抱え込むなんてことは
ないので, どんどん追加, 修正していけばよいと思います.

わかりました。emacs-jpの場合聞かないでよさそうなものそのままpushしていきます。ありがとうございます。

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