---
# System prepended metadata

title: '今週何知った？ week:40'

---

## 各自発表

> [name=ken3ypa]

## 概要
- 会社で使っているgemをぼちぼち読んでいる
- 読んでいると、小さいコントリビュートネタがそれなりに見つかる。と同時に、知ってる人がいくつか小さいPRを出している
- どのようなPRを出しているかをいくつか眺めてみる

### r7kamuraさん
- [Fix callback order for carrierwave 2.0 by r7kamura · Pull Request #297 · lardawge/carrierwave_backgrounder](https://github.com/lardawge/carrierwave_backgrounder/pull/297)
    - carrierwave側の変更に伴い、carrierwave_backgrounder
    - carrierwave 2.0 から、after_save コールバックにフックされていたものが、after_commit にフックされるようになった
- [Improve gemspec URLs by r7kamura · Pull Request #7 · masa16/holiday_japan](https://github.com/masa16/holiday_japan/pull/7)
    - gemspec 内の記述追加・変更

### a_matsudaさん
- [No need to send a public method "Object#extend" by amatsuda · Pull Request #518 · stefankroes/ancestry](https://github.com/stefankroes/ancestry/pull/518)
    - Object#extend はパブリックメソッドなので send する必要なし
- [GitHub is HTTPS by default by amatsuda · Pull Request #296 · magnusvk/counter_culture](https://github.com/magnusvk/counter_culture/pull/296)
    - http -> https
- [Lazy-load ActionMailer::Base to defer :action_mailer on_load hook by amatsuda · Pull Request #66 · fgrehm/letter_opener_web](https://github.com/fgrehm/letter_opener_web/pull/66)
    - ActionMailer::Base を直に触るのではなく、ActiveSupport on_load hook 経由で呼んで

### onkさん
- [Remove dependency to after_commit_action gem by onk · Pull Request #297 · magnusvk/counter_culture](https://github.com/magnusvk/counter_culture/pull/297)
    - 依存が削除されたはずの gem の記述が残っている

### tricknotesさん
- [Add changelog for v2.4.3 by tricknotes · Pull Request #505 · rubysherpas/paranoia](https://github.com/rubysherpas/paranoia/pull/505)
    - changelog がなかったから足した
- https://github.com/rubysherpas/paranoia/pull/506
    - .gemspec の記述が古い
- [[DON'T MERGE] Add failing tests for association with abort on destroy by tricknotes · Pull Request #522 · rubysherpas/paranoia](https://github.com/rubysherpas/paranoia/pull/522)
    - おそらく意図していないであろう挙動を報告し、期待する動作とそれを検証・担保するテストを提供
- [Test against Ruby 2.3.0 on Travis CI by tricknotes · Pull Request #54 · amatsuda/active_decorator](https://github.com/amatsuda/active_decorator/pull/54) & [Drop test for Ruby 1.8 & 1.9 by tricknotes · Pull Request #46 · amatsuda/active_decorator](https://github.com/amatsuda/active_decorator/pull/46)
    - CIでカバーするRubyのバージョンを足したり消したり
- [Disable Launchy and ignore executable command detection by tricknotes · Pull Request #75 · fgrehm/letter_opener_web](https://github.com/fgrehm/letter_opener_web/pull/75)
    - launchy のコマンドが検出できなかった場合に例外を吐くが、その場合環境変数が DRY_RUN = 'true' のままになってしまうのを修正

### y_yagiさん
- [Support Rails 6.1 by y-yagi · Pull Request #111 · amatsuda/active_decorator](https://github.com/amatsuda/active_decorator/pull/111)
    - RailsのPartialRendererに対してリファクタが行われたため、新しいレンダリングのふるまいにあわせた修正をおこなった。
    - すき
        - @y-yagi Hi! Glad seeing you here, and thank you so much for the perfectly crafted patch! ❤️
- [Fix a method name typo in CHANGELOG by y-yagi · Pull Request #46 · veeqo/activejob-uniqueness](https://github.com/veeqo/activejob-uniqueness/pull/46)
    - README 記載のメソッド名が間違っていた

---

> [name=makicamel]
