Ce diaporama a bien été signalé.
Nous utilisons votre profil LinkedIn et vos données d’activité pour vous proposer des publicités personnalisées et pertinentes. Vous pouvez changer vos préférences de publicités à tout moment.

プルリク送ったらライブラリ壊したった

99 vues

Publié le

puプルリクを送ったらライブラリを壊してしまった話

Publié dans : Logiciels
  • Soyez le premier à commenter

  • Soyez le premier à aimer ceci

プルリク送ったらライブラリ壊したった

  1. 1. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0 プルリク送ったら ライブラリ壊し たった 鍋太郎
  2. 2. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0 自己紹介 プログラマー✓ Debian JP 事務局✓ 小市若者連 委員長 ←New✓ https://about.me/nabetaro✓
  3. 3. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0 Phony 電話番号を扱う✓ 世界中が目標✓ E164準拠✓
  4. 4. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0 使い方 Phonyモジュールのメソッド normalize 正規化✓ format 整形✓ etc...✓ ✓ Phony['国番号'] その国の電話番号が使える…はず✓ ✓
  5. 5. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0 使ってみる Phony.normalize('+81 3-1234-5678') => "81312345678" Phony.normalize('03-1234-5634', cc: '81') => "81312345634" Phony["81"].normalize('+81 3-1234-5634') => "81312345634" Phony["81"].normalize('03-1234-5634') => "312345634" # !?
  6. 6. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0 使ってみる Phony.format('81312345634', cc: '81', format: :international) => "+81-3-1234-5634" Phony.format('81312345634', cc: '81', format: :national) => "03-1234-5634" Phony.format('81312345634', cc: '81', format: :local) => "1234-5634" Phony['81'].format('0312345634', format: :international) => "+81-03-1234-5634" # !? Phony['81'].format('0312345634', format: :national) => "003-1234-5634" # !? Phony['81'].format('0312345634', format: :local) => "1234-5634"
  7. 7. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0 犯行動機 国内で利用している電話番号も 使えるようにしたかった。 '0312345634' を 03-1234-5634 に✓ ✓
  8. 8. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0 犯行内容 テストケースを書いてプルリク 送った ✓ https://github.com/floere/phony/pull/453 2日で修正された
  9. 9. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0 結果 Phony.format('81312345634', cc: '81', format: :international) => "+81-3-1234-5634" Phony.format('81312345634', cc: '81', format: :national) => "03-1234-5634" Phony.format('81312345634', cc: '81', format: :local) => "1234-5634" Phony['81'].format('0312345634', format: :international) => "+81-3-1234-5634" Phony['81'].format('0312345634', format: :national) => "03-1234-5634" Phony['81'].format('0312345634', format: :local) => "1234-5634" ええやん
  10. 10. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0 結果 Phony.normalize('+81 3-1234-5678') => "81312345678" Phony.normalize('03-1234-5634', cc: '81') => "810312345634" # !!! Phony["81"].normalize('+81 3-1234-5634') => "81312345634" Phony["81"].normalize('03-1234-5634') => "0312345634" まって、まって、まって
  11. 11. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0 影響 phony_rails rails向けライブラリ✓ 電話番号の正規化✓ 電話番号のvalidation✓ ✓ https://github.com/joost/phony_rails おかしくなっとる!!!
  12. 12. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0 自分の取った対応 追加テストケースのプルリク https://github.com/floere/phony/ pull/456 ✓ 未マージ✓ ✓ revertのプルリク https://github.com/floere/phony/ pull/457 ✓ ✓
  13. 13. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0 おすすめ対応 2.18.10 で止めておく✓ プルリク #457 を適用✓
  14. 14. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0 考察 Phony["81"].normalizeの結 果の先頭に国番号を付加 ✓ 日本の電話番号は 国内プレ フィックス「0」を取って国番 号を付加 ✓
  15. 15. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0 余談 市外局番には国内プレフィック ス(trunk)は含まれない https://en.wikipedia.org/wiki/ Telephone_numbers_in_Japan Talkページで先頭の0について議論してる✓ わかりにくい…✓ ✓ ✓
  16. 16. プルリク送ったらライブラリ壊したった Powered by Rabbit 3.0.0 まとめ オチはありません。✓ 良かれと思ってもうまくいかな いこともあります。 ✓ 適切な改善案を出しましょう。✓ おちこんだりもしたけれど、私 はげんきです。 ✓

×