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

Support number format option. (#11) #13

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

masa-kunikata
Copy link

再度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]

@coveralls
Copy link

Coverage Status

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

@sugi
Copy link
Owner

sugi commented Nov 9, 2019

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

@sugi
Copy link
Owner

sugi commented Dec 23, 2019

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

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

@sugi
Copy link
Owner

sugi commented Dec 23, 2019

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

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