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

Update help #5

Merged
merged 3 commits into from
Jul 30, 2016
Merged

Update help #5

merged 3 commits into from
Jul 30, 2016

Conversation

akinomyoga
Copy link
Collaborator

すみません。

  • さきの変更で show_help に -I を追記するのを忘れていました。
  • また、README.md の更新を忘れていました。
  • help で comma のあるオプションとないオプションがあって気になっていたので comma を挿入しました。

よろしければお願い致します。

@coveralls
Copy link

coveralls commented Jul 30, 2016

Coverage Status

Coverage remained the same at 52.076% when pulling 313ca7b on akinomyoga:fix-help into 0ac0fd6 on saitoha:master.

@saitoha saitoha merged commit 58b75fc into saitoha:master Jul 30, 2016
@akinomyoga
Copy link
Collaborator Author

akinomyoga commented Jul 30, 2016

ありがとうございます!

あと、この PR とは関係ない事なのですが…

GitHub 上で、Issues が無効になっています。uobikiemukot/recterm から fork したときのデフォルトの設定になっているためだと思うのですが、問題点や要望などがあったばあいに報告がしやすくなるので Issues が有効になっていたほうがこちらとしては気楽でございます。いかがでしょう。(Issues の有効化は Repository の Settings - Options - Features - Issues のチェックボックスからできるようです。)

@saitoha
Copy link
Owner

saitoha commented Jul 30, 2016

@akinomyoga なるほど、ご指摘ありがとうございます!
Issue有効にしておきました。あと、貢献をして頂いているので、Collaboratorsにも招待しておきました。

@akinomyoga
Copy link
Collaborator Author

@saitoha ご招待ありがとうございます! もし何か貢献できることがあればさせていただきたいという思いで参加させて頂きます。ただ、すみません、Collaborator というものは初めてなので今どういうものなのか (何ができるか・するべきか・しないべきか, etc.) 調べているところです…何か要望などありましたらどうぞ仰って下さい。よろしくおねがいいたします。

@saitoha
Copy link
Owner

saitoha commented Jul 30, 2016

@akinomyoga すみません、逆に混乱させてしまっていますね。
コミット権がある状態なので、ヘルプやgh-pagesの更新などは、普通にマージして頂いて構いませんです。
仕様や実装について話し合いながら進めたいときは、これまでどおりpull-requestベースでいきましょう。

@akinomyoga
Copy link
Collaborator Author

@saitoha ありがとうございます。なるほど。調べると、事前に色々取り決めておかなければ work しない仕組みのように感じられて、不思議に思っていた所でした。細かいことは除き、今迄どおりにさせていただくことにします。

@akinomyoga akinomyoga deleted the fix-help branch July 30, 2016 15:54
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.

None yet

3 participants