2018-01-01から1年間の記事一覧

1回休み

activerecord-import で Factory Girl のまま Factory Bot に置き換えられていなかったのを PR したくらいで1回休み。

Layout/EmptyLinesAroundAccessModifierとLayout/EmptyLinesAroundClassBodyの組み合わせで起きることがあるinfinite loopを直した

github.com 問題の再現から要因の特定に2時間くらい、アクセス修飾子の前後についての現状実装で、正規表現で空白の有無が妥当かどうかを調べているあたりどうしたものか悩むのに2時間越えくらい。解決するための実装アプローチとして、正規表現から AST に…

Ginza.rb 第63回

『Ginza.rb 第63回 非常識な速さ dev.to の秘密を探れ』に行った。会場は銀座のメドピアさん。 ginzarb.doorkeeper.jp 当日は y-yagi さん謹製の Gist をベースに進んだ。 https://gist.github.com/y-yagi/d166b23a5f652c7826d9c47f3a49b858 印象に残ったの…

`Layout/EmptyLineAfterGuardClause` cop のヒアドキュメントまわりの偽陽性を直した

RuboCop 0.59.1 でのリグレッションのフィードバックをもらった。 github.com 以下のような後置でない条件でヒアドキュメントを含むケースについて offense を発生させる偽陽性を含んでいた。 def method if truthy raise <<-MSG This is an error. MSG end …

アサインガチャ1回当たりで、1回休み

先日 Cucumber の CI を見てみる際に気に掛かった nitpick を PR したら、アサインガチャで自分が当たってなるほど?だった。 github.com 1回当たりで、1回休み。

RuboCop 0.59.1 がリリースされた

RuboCop 0.59.1 がリリースされた。今回はパッチバージョンということで、RuboCop 0.59.0 を使っているユーザーは上げておくのがおすすめ。 github.com ざっくりと今回の変更履歴の内容を取り上げる。 Bug fixes #6267: Fix undefined method 'method_name' …

5xRubyの竜堂さんたちと食事した

来日されていた 5xRuby の竜堂さんたちと晩御飯をご一緒した。テックトークから、台湾での流行りの技術から今期の推しアニメまで歓談しつつ過ごしたりしていた。コミュニケーションを取る際に、英語のターンで聞き取りが課題だなあと人生何度目かの意識の高…

1回休み

パッチ会の次回予告と今回まとめを書いたり 、RuboCop に出していたパッチを見直したり、CI の落ちていた RuboCop Markdown に PR を出したりしていた。

Rails/OSS パッチ会 (2018年9月)

Rails / OSS パッチ会だった。 ちょうど RuboCop の CI が Travis CI から CircleCI に移行したタイミングで、ruby-head のデイリービルドイメージがなくてどうしたものか mtsmfm に相談したところ自前で用意しているとのことだった。Official Docker image …

Asakusa.rb 第478回

Asakusa.rb に行った。ちょうど一昨日リリースされた RuboCop 0.59.0 でいくつかイシューのフィードバックがあったので、そのバグフィックスをしたりしていた。 再現テストから副次的なバグも見つかって、一方のテストをパスしたら他方のテストが落ちるとい…

`Style/EmptyCaseCondition` copのautocorrectでコメントがなくなるケースを直した

少し前にフィードバックをもらっていたイシューとなる。 github.com 以下のように、case と最初の when の間にあるソースコードコメントが autocorrect で消えるというフィードバックだった。 def foo case # when bar ... when bar 2 end end プロダクトコ…

RuboCop 0.59.0 がリリースされた

今回のリリースは 0.58.2 以前からのバグフィックスが多めだと思う。自分の変更の中では Changes セクションにあるものが影響の大きめなものだと捉えている。 github.com 自分の変更に関するコメント。 New features #5659: Make Layout/EmptyLinesAroundCla…

1回休み

builderscon 帰りの onk と飲んで、builderscon がどうだったとか聞いたり近況を話したりしていた。

累の最終巻を読んだ

最終巻がリリースされていた。読み始めたら続きや結末が気になって最後まで読んでいた。 累(1) (イブニングコミックス)作者: 松浦だるま出版社/メーカー: 講談社発売日: 2014/01/31メディア: Kindle版この商品を含むブログ (2件) を見る

表参道.rb #38

「表参道.rb #38 〜Railsアンチパターン〜」で発表してきた。会場は外苑前のビジネスバンクグループさん。 omotesandorb.connpass.com 一般参加者枠が満席だったので、当日直前まで発表者枠が空いていたら参加しようと思っていたところ、そうなったのでそう…

RuboCopが静的解析のRubyバージョンを決める流れ

RuboCopが対象のRubyバージョンを決める方法は、探索順位によるため期待どおりランタイムの Ruby のバージョンと一致している場合以外に、条件によってはそうでないこともある。 .rubocop.yml の TargetRubyVersion の値、.ruby-version の値、Gemfile.lock …

Rails 5.2でActive RecordからArelのメソッドへの移譲に追加されている非推奨警告

Rails 5.2.1 アプリケーションの 24 Pull Requests へのパッチを元にした、昨年末の日記の関連。 以下のような警告の川が流れた際には、.arel をレシーバーとして明示的に指定する。 DEPRECATION WARNING: Delegating exists to arel is deprecated and will…

るろうに剣心 北海道編 (1) を読んだ

剣心華伝か何かに書かれいてた後日談構想の実装ということで次巻以降も読んでいくと思う。 るろうに剣心─明治剣客浪漫譚・北海道編─ 1 (ジャンプコミックス)作者: 和月伸宏,黒碕薫出版社/メーカー: 集英社発売日: 2018/09/04メディア: コミックこの商品を含…

`Layout/EmptyLineAfterGuardClause` cop をデフォルトで有効にした

数日前の話だが Layout/EmptyLineAfterGuardClause cop をデフォルトで有効にする PR を開いてマージされた。 github.com RuboCop の次のリリース (たぶん 0.59.0) で、ガード条件の後ろに空行を入れる cop がデフォルトで有効ということになる。 auto-corre…

リファクタリング 第2版が素晴らしい

同僚の haru01 のリツイートで『リファクタリング 第2版』のカットオフを読めると知って、Safari online books service で斜め読みした。同サービスへのフリートライアルでも読むことができる。 If you have a subscription to the Safari online books serv…

RSpec 3.8.0 で Change マッチャーの細かな振る舞いが変わった

RSpec 3.8.0 で RSpec::Matchers::BuiltIn::Change マッチャーによる判定に Object#hash も使われるようになった。 以下の PR での変更となる。 github.com ピンポイントで示すと以下の Change#changed? メソッドの実装で、変更前に保持している hash 値と変…

FactoryBot 0.4.11 での非推奨警告を抑える

FactoryBot 0.4.11 で closed_at 1.day.from_now といった FactoryBot のコードに対して以下のような警告が出るようになった。 DEPRECATION WARNING: Static attributes will be removed in FactoryBot 5.0. Please use dynamic attributes instead by wrapp…

1回休み

RuboCop の方は、CI を CircleCI にする動きが本格化していて、色々と動きが混ざってもなんなので一休み。ローカルリポジトリにある Gem で警告が出るものにパッチを送るなどしていた。

JOJO 第4部を読み終わった

上京する前後が第4部だった気がして、最後まで読んだかあやしいパートでいろいろと復習できてよかった。 ジョジョの奇妙な冒険(第4部) ダイヤモンドは砕けない 文庫版 18-29巻セット (化粧ケース入り) (集英社文庫)作者: 荒木飛呂彦出版社/メーカー: 集英社…

iPhone X を修理しに行った

一週間くらい前から、数十秒起動していると右上が緑色に発光するという現象になって、液晶パッドがタッチしていない状態で過度に反応したり、反対にまったく反応しない箇所が現れたりしていて不便していた。 ソフトバンクでは受け付けていないとのことで、ジ…

1回休み

JOJO 第4部を再読こみで読み始めた。 ジョジョの奇妙な冒険(第4部) ダイヤモンドは砕けない 文庫版 18-29巻セット (化粧ケース入り) (集英社文庫)作者: 荒木飛呂彦出版社/メーカー: 集英社発売日: 2006/05/22メディア: 文庫購入: 21人 クリック: 496回この商…

1回休み

SSIA.

Code Review Meetup #4 で話します

『Code Review Meetup #4 Code Reviewで集まろう』の登壇タイトルとコンテンツが公開された。 sideci.connpass.com コードレビューそのものに関する話はあまりしてこなかったので、そういった話を盛り込んで行く予定。 タイトルは onk の『RSpec しぐさ』へ…

Style/EmptyCaseConditionのauto-correct不具合を避けるようにした

以下のイシューで Style/EmptyCaseCondition に関する auto-correct の不具合のフィードバックがあったことがはじまり。 github.com ちょうど今日、勤務先での月例会の後に残ったメンバーで、上記イシューにおける case...when と if...elsif での振る舞いの…

Railsdm Podcast #2 を聴いた

Railsdm Podcast #2 - It's self-evident (@mtsmfm) を聴いた。 公開しました - #2 - It's self-evident (@mtsmfm) by Railsdm Podcast #railsdm https://t.co/tr3vcPtC1c— 徳用カルパス (@yoshi_hirano) August 21, 2018 ちょくちょく「koic さん」と言及し…