Weekly Report 2020/10/11
週報3回目です。毎週日曜日に更新するつもりでしたが早速すっぽかしました(2回目)。
今週は deno_lint 中心で作業しました。
denoland/deno_lint
Pull Request
- fix(prefer-const): refactor & handle hoisting
- 以前自分が実装した
prefer-constルールにおいて、変数の巻き上げに対応できていなかったのを修正したPRです。たかが変数の巻き上げに対応するだけなのですが、実装をほぼまるっと変える必要があって、かなり大変でした。テストも含めてですが、差分が +1,381 −418 なので僕が今まで OSS に出した PR の中では最大かもしれないです。
- 以前自分が実装した
- chore: enable prefer-const again
prefer-constが今度こそ完成したのですが、recommended rules (Deno 本体から使われるルールセット) に入れ直すのを忘れていました。入れるだけのPRです。
- refactor: remove Arc/Rc
- コードを眺めていて「この
Rc/Arcいらなそうだなぁ」と思ったところがあったので、これらを使わないように書き換えてみました。Rc/Arcは不要になったものの、代わりにVecのアロケーションが発生したりして、効率向上につながったかは謎ですが、approve されたので良しとします(それでいいのか)
- コードを眺めていて「この
- refactor: use matches!
- おそらく Rust 1.47 から、
matches!マクロを使えるところで使っていないと clippy が文句を言ってくるようになりました。その対応と、ついでに GitHub Actions 上で動く Rust のバージョンアップおよび README の修正を行いました。
- おそらく Rust 1.47 から、
- fix(getter-return): use ControlFlow to check if getters return something
- こちらの issue で報告されている false positive を修正すべく、
Control Flowと呼ばれている制御構造の解析器をいじったPRです。最初Control Flowの解析がどのように動いているのかまったく分からず、いろいろテストを追加しながらやってたらまた大きめの PR になってしまいました。レビュー待ちです。
- こちらの issue で報告されている false positive を修正すべく、
雑記
RSLint が爆速でスターを伸ばしていて、あっという間に deno_lint のスター数を追い抜きました。どこかでバズったりしてるのかな？ Reddit でかなり upvote を集めているのはみかけました。
RSLint のロードマップなどちらっとチェックしていますが、すごく計画性があるというか、プロジェクトとして大成させるための長期的な見通しが明確に見えているのだなというのが伝わってきてすごいです。あと内部実装も行き当たりばったりというわけではなく、すべてを理解しながら書かれているとってもきれいで整理されたコードという印象を抱きました。
今後もしっかりウォッチしていって、コントリビュートもしていければと思います。