[tests]: Set modules in all plot_* calls within test functions
メンテナーはふだん 1 日以内に返信
@Schiano-NOAA がすでに取り組んでいます。
2026年10月13日 から。
評価
調査の方向性
ペイロードにはテストファイル名が記載されていないため、まずローカルでテストスイートを実行し、対話型のモジュールプロンプトが表示される箇所を確認してください。次に、module引数を省略しているテスト関数内のplot_*呼び出しを探します。それぞれの呼び出しに有効なmoduleの値を追加してください。値の選択はissueで未決定のため、プロット関数がその値をどのように使っているかを確認してください。完了条件は、スイートがプロンプトなしで実行されることと、任意のfilter_dataのテストがユーザー選択のパスをカバーすることです。
索引モデルが issue の本文から書いたものです。
説明
Presently, all tests of the plotting function omit the module parameter, which eventually prompts the user to select a module via the R terminal. This makes running the tests suite locally not automatic (the user has to continuously interact with the tests by selecting modules). It would probably be more efficient to set the module parameter in each call to a plot_* function within the testing suite, rather than leaving it to wait for a user prompt.
An additional test could be created specifically for filter_data that tests that the user selection works, if there are concerns about that functionality breaking.
- 主要言語
- R
- スター
- 20
- フォーク
- 9
- 平均マージ
- 1日 5時間
- マージ済み PR(30日)
- 12
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
nmfs-ost/stockplotr のほかの issue
-
難易度 5/5 1週間以上 初心者へのやさしさ 15/100
nmfs-ost/stockplotr#422 ·
メンテナーはふだん 1 日以内に返信
-
[fix]: module name selection retained in environment when `make_rda = T`対応中かも @sbreitbart-NOAA が 1 日前に担当しました。 オープンbug
nmfs-ost/stockplotr#421 · コメント 5 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Make tests more efficient対応中かも @JZahner1 が 1 日前に担当しました。 オープン
nmfs-ost/stockplotr#420 · コメント 4 件 · 担当者 2 名 ·
メンテナーはふだん 1 日以内に返信
-
[tests]: are "rda file made when indicated" tests needed for every plot?対応中かも @Schiano-NOAA が 2 日前に担当しました。 オープンquestion
nmfs-ost/stockplotr#418 · コメント 10 件 · 担当者 3 名 ·
メンテナーはふだん 1 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 40/100
nmfs-ost/stockplotr#415 ·
メンテナーはふだん 1 日以内に返信
nmfs-ost/stockplotr の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
mitchelloharawild/ggtime#39 ·
-
enhancement good first issue help wanted
難易度 2/5 半日 初心者へのやさしさ 82/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
-
question
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
-
Breaking 🏴☠️ Consistency :green_apple: :apple:
難易度 1/5 1時間未満 初心者へのやさしさ 70/100
easystats/bayestestR#789 ·
メンテナーはふだん 1 日以内に返信