wareki icon indicating copy to clipboard operation
wareki copied to clipboard

Support number format option. (#11)

Open masa-kunikata opened this issue 6 years ago • 4 comments

再度PRです issue #11 の件、下記仕様の”数値”部分のみを参考にしてやってみました

https://docs.ruby-lang.org/ja/latest/method/Time/i/strftime.html

このメソッドは strftime(3) や glibcの仕様を参考に作成されており、以下のオプションが利用できます。

^: 大文字で出力を行なう #: 小文字であれば大文字に、大文字であれば小文字に変更する -: 左寄せにする(0埋めや空白埋めを行わない) _: 空白埋めにする 0: 0埋めにする 数値: 表示桁数を指定する


以下、お伝えしたい点

  • 大文字/小文字 の変換は不要と思ったので考慮しませんでした。

  • 漢数字出力時は、おそらく不要なんではないかと思ってやってません。

  • 標準の Date#strftime の %m %d 指定になんとか準拠したつもりではあるんですが、本当にあっているのか不安ではあります、、、

  • format メソッドの引数 opt を追加する形にしてみたんですが、おそらくこの影響でRubocopに怒られてしまいました

Metrics/AbcSize: Assignment Branch Condition size for format is too high. [101.2/99]

masa-kunikata avatar Jun 16 '19 01:06 masa-kunikata

Coverage Status

Coverage increased (+0.2%) to 94.649% when pulling b19d0d9761a12f7c79eff8ca956b4286eae74259 on masa-kunikata:master into d197ab77e5a4625eabba569423b88b5f82ff2daf on sugi:master.

coveralls avatar Jun 16 '19 01:06 coveralls

ぎゃーーーすみません! こちら、はるか前で物凄い申し訳ないのですが見落としていました。 月末になってしまいますが確認します! :bowing_man:

sugi avatar Nov 09 '19 07:11 sugi

うーむむむむむむ。すみません、これを今まで見られていなかったのが大変申し訳ない……。 良さそうに見えます。

rubocop に関しては、そこまで厳しくする必要は無いので調整します。

sugi avatar Dec 23 '19 17:12 sugi

ちょっとこれを取り込ませて頂くのと、 _parse の日本語サポートを入れた状態で、非互換もあるので 2.0 として出そうと思います。 もう暫くお待ちください……

sugi avatar Dec 23 '19 17:12 sugi