OSS Gate Workshop: techouse: 2024-04-18: izumitomo: mechanize: Work log

[izumitomo]

This is a work log of a "OSS Gate workshop".
"OSS Gate workshop" is an activity to increase OSS developers.
Here's been discussed in Japanese. Thanks.

作業ログ作成時の説明

以下のテンプレートを埋めてタイトルに設定します。埋め方例はスクロールすると見えてきます。

OSS Gate Workshop: ${LOCATION}: ${YEAR}-${MONTH}-${DAY}: ${ACCOUNT_NAME}: ${OSS_NAME}: Work log

タイトル例↓：

OSS Gate Workshop: Tokyo: 2017-01-16: kou: Rabbit: Work log

OSS Gateワークショップ関連情報

• スライド：ワークショップの進行に使っているスライドがあります。
• チャット：OSS開発に関することならなんでも相談できます。ワークショップが終わった後もオンラインで相談しながら継続的にOSSの開発に参加しましょう！
• シナリオ：ワークショップの目的・内容・進め方の詳細が書いています。
• 過去のビギナーの作業ログ：他の人の作業ログから学べることがいろいろあるはずです。

[izumitomo]

業務でMechanizeというgemを使っているが、ローカルで立ち上げで1から構築して動かしてみる、というのをやったことがなかったのでやってみる。

[izumitomo]

GitHubのRepositoryを確認
https://github.com/sparklemotion/mechanize?tab=readme-ov-file

MIT licenseであることを確認

[izumitomo]

とりあえずREADMEを上から順に読んでいくことにする。
冒頭からリンク切れをしている。

[izumitomo]

README
Descriptionを読んで、なんとなくMechanizeってcookieを保存したり、遷移の履歴を保存できるんだなあ、という所感。
これがdescriptionなのか？　という疑問を抱く。

[izumitomo]

README
Dependencyを読んで、色んなgemを使っていることを把握。

[izumitomo]

README
exampleとしてGUIDE.rdocなるものがあるらしいので、それを読んでいくことにする

[izumitomo]

exampleを読む前にリンク切れの件を調べてみる。
切れているリンク
https://github.com/sparklemotion/mechanize/actions/workflows/ci-test.yml
テスト結果のリンクを指していたと思われるので、actionを見てみると、
https://github.com/sparklemotion/mechanize/actions/workflows/ci.yml
テスト結果を示す物を発見。

[izumitomo]

先月、github actionを走らせるactionを定義したファイル名を
ci-test.yml -> ci.yml
と、変更しているコミットがある。
sparklemotion/mechanize@f89dc84

しかし、READMEのリンクを併せて変更することを忘れていて、リンク切れを起こしている。

[izumitomo]

今から修正のPRを作成する。

[izumitomo]

READMEを編集ボタンを押したら、Forkしますか的なボタンが出てきたので押した。
（それ以外に動線がなかったので押すしかなかった）

ボタンを押した先のリンク先のスクショ

[izumitomo]

エディタ上で正しいURLに修正。
Commit changeを行うと、forkしたRepositoryにpatch-1というブランチが作られた。
patch-1というブランチ名をfix-READMEに変えた。

[izumitomo]

PRを作成
sparklemotion/mechanize#643

[izumitomo]

gimeiのgem を調べてみる。

[izumitomo]

README

Gimei.unique.family で生成された名前は Gimei.unique.clear(:last) で消去します。 また、Gimei.unique.given で生成された名前は Gimei.unique.clear(:first) で消去します。

直感的には、Gimei.unique.family で生成された名前は、Gimei.unique.clear(:family)で削除させた方がいいのではないか？

[izumitomo]

willnet/gimei#50

このPRでfamilyとgivenが追加され、生成時のaliasが定義されている。
ただ、clearの処理は後回しにしたのか、他に理由があったのか、aliasを追加していない。

[izumitomo]

修正はできたが、レポジトリをクローンしても、レポジトリ内のテストが実行できない。
Gemfileにtest用のgemが何も記述されていない。
rubyのバージョンが記述されていない。

[izumitomo]

そもそもGemfileの修正が必要なので、PRは複数作る必要ありそう。

[github-actions]

おつかれさまでした！

ワークショップの終了にともないissueを閉じますが、このまま作業メモとして使っても構いません 👌

ワークショップの感想を集めています！

ブログなどに書かれた際は、このページへリンクの追加をお願いします 🙏

またの参加をお待ちしています！