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

#76: Read markdown content from std input #77

Merged
merged 4 commits into from Nov 26, 2019
Merged

Conversation

@515hikaru
Copy link
Contributor

515hikaru commented Nov 23, 2019

Closes #76

tj/commander.js を使って -s または --stdin オプションで標準入力を読み込めるようにしました。

@515hikaru

This comment has been minimized.

Copy link
Contributor Author

515hikaru commented Nov 23, 2019

commander.js を利用したことで、自動で -h または --help オプションが使えるようになっています。

引数の名前などを設定していないので、現在の help の出力はこんな感じです。

Usage: index [options]

Options:
  -s, --stdin  read standard input
  -h, --help   output usage information

help出力をよりわかりやすくすることは今回のスコープからははずれると思ったので、現時点では対処していません。

@37108 37108 self-requested a review Nov 25, 2019
Copy link
Owner

37108 left a comment

対応ありがとうございます。こちらのレビューが遅くなってしまい申し訳ありません。
2点だけ変更をお願いします。

package.jsonのバージョンを0.6.0にあげて貰ってもよろしいでしょうか。

"version": "0.5.6",

importの順序をmd2bgが一番下でかつ、上の3つとの間に空行があると嬉しいです。

import * as fs from 'fs'
import * as path from 'path'
import { md2bg } from '../'
import { Command } from 'commander'

@37108
37108 approved these changes Nov 25, 2019
Copy link
Owner

37108 left a comment

対応ありがとうございます。こちらのレビューが遅くなってしまい申し訳ありません。
2点変更をお願いします。

package.jsonのバージョンを0.6.0にあげて貰ってもよろしいでしょうか。

"version": "0.5.6",

importの順序をmd2bgが一番下でかつ、上の3つとの間に空行があると嬉しいです。

import * as fs from 'fs'
import * as path from 'path'
import { md2bg } from '../'
import { Command } from 'commander'

@37108 37108 self-requested a review Nov 25, 2019
@37108

This comment has been minimized.

Copy link
Owner

37108 commented Nov 25, 2019

@515hikaru
GitHub側でBadGateway出た後に再レビューをした結果として同一コメントでRequested ChangesとApprovedが出てしまいました。申し訳ありません。

前者のRequested Changesが正しく変更箇所の修正をお願いします。

@515hikaru

This comment has been minimized.

Copy link
Contributor Author

515hikaru commented Nov 25, 2019

@37108 修正しました。

@37108
37108 approved these changes Nov 26, 2019
Copy link
Owner

37108 left a comment

LGTM

@37108 37108 merged commit a789941 into 37108:master Nov 26, 2019
@515hikaru

This comment has been minimized.

Copy link
Contributor Author

515hikaru commented Nov 26, 2019

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.