-
Notifications
You must be signed in to change notification settings - Fork 547
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OSS Gate Workshop: ゆるい勉強会旭川: 2024-07-28: satou-haruka-37: ransack: Work log #1858
Comments
https://github.com/activerecord-hackery/ransack を選びました。 https://github.com/activerecord-hackery/ransack/blob/main/LICENSE と、MITライセンスがあったので、OSSだと確認できました。 |
サポーターよりOSS contributeのアイデアの助言 |
日本語とカタカナを同じものとみなして検索する方法 |
サポーターより助言 |
https://github.com/activerecord-hackery/ransack/blob/main/CONTRIBUTING.md#pull-requests |
実行したらエラーが出た
|
抜粋
|
サポーターより助言
|
コントリビューションガイド通りにやってるのにセットアップができないと言うことがわかった。 |
下記のように実行して
|
次の段階に移る
実行したらテストが通らなかった。
|
Pending: (Failures listed here are expected and do not affect your suite's status)
|
spec/ransack/search_spec.rbの183行目
で、
|
こちらは Pending だが、他の Failures も同じように |
|
|
ググって見つかったもの |
|
その後何から手をつけたら良いか分からずとりあえず
|
RubyやRailsのバージョンが関係しているのかと思い確認した。
|
報告1つ目
調べたところ記事によると以下を実行すると良いと言うことだったので、
実行後、再度 |
報告2つ目
を実行したところ、テストが通らず以下のエラーになったことを報告します。
|
#1858 (comment) 日本語
英語
|
サポーターの助言より もし macOS ユーザー( homebrew )を使用していて、以下のエラーが出た場合には
環境変数ライブラリエクスポートに zstd のパスを設定する必要があります。
|
If you are a macOS user ( homebrew ) and you get the following error
You need to set the path to zstd in the environment variable library export.
|
プルリクの文面 Contributing to Ransack の通りに
そのため、注釈を追記するのがベターでは無いかと考えました。 |
As per Contributing to Ransack
Therefore, I thought it would be better to add an annotation. |
報告2つ目 I was proceeding as described in Contributing to Ransack,
command, the test did not pass,
|
現時点で報告するならこれで、 |
2つ目で報告しようとしていたものが既に他の人が報告していた。 さらにその返信で、既に対応していた。(マージは未) |
サポーターより助言 |
ransack の元のリポジトリのプルリクを、
|
実行結果
|
|
この状態で再度テストを走らせたところ、Pending以外全て通った。
|
このことをプルリクエストにコメントする。 |
のコマンドを実行すると、私も他の方と同様のエラーが発生し、テストが通りませんでした。
command, I get the same error as the others and the test does not pass.
|
このプルリクエストを
Bring this pull request locally using
|
実際に送ろうと思うコメント |
I saw the same thing in the main branch.
I applied this pull request locally and got it to work, and it passed the test.
Below is the execution log.
|
いつ誰がやっても実行しやすい方のコマンド(他の人が再現しやすい)
|
コメントした。 |
今回は本来コントリビュートするべき手前の段階でエラーになってしまった。本来であればこの段階を通過した後どのようなコントリビュートをする想定だったのか? →これは想定通り。これが今回の回でやりたかった意図通りのこと |
おつかれさまでした! ワークショップの終了にともないissueを閉じますが、このまま作業メモとして使っても構いません 👌 ワークショップの感想を集めています! ブログなどに書かれた際は、このページへリンクの追加をお願いします 🙏 またの参加をお待ちしています! |
コメントしたプルリクエストがクローズされた。 activerecord-hackery/ransack#1506 (comment) おそらく以下のプルリクエストで修正されている。 |
別件で立てたこちらのプルリクエストはまだ反応なし。 |
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ワークショップ関連情報
The text was updated successfully, but these errors were encountered: