Skip to content

[GRDM-59575 Workaround] BinderHubアドオン repo2docker テストに remotes を追加#40

Merged
yacchin1205 merged 1 commit into
RCOSDP:mainfrom
yacchin1205:fix/binderhub-r-remotes
May 26, 2026
Merged

[GRDM-59575 Workaround] BinderHubアドオン repo2docker テストに remotes を追加#40
yacchin1205 merged 1 commit into
RCOSDP:mainfrom
yacchin1205:fix/binderhub-r-remotes

Conversation

@yacchin1205
Copy link
Copy Markdown
Collaborator

@yacchin1205 yacchin1205 commented May 25, 2026

Purpose

テスト手順-BinderHubアドオン-repo2docker.ipynb / テスト手順-BinderHub-BinderHubアドオン-repo2docker.ipynb / テスト手順-BinderHubアドオン-アドオン追加.ipynb で、解析環境ビルド時に remotes パッケージを事前に R 依存として登録しないとビルドに失敗するケースがある。各テスト手順で R パッケージを事前登録できるようにし、デフォルトに remotes を含める。

Changes

  • テスト手順-BinderHubアドオン-repo2docker.ipynb および テスト手順-BinderHub-BinderHubアドオン-repo2docker.ipynb
    • パラメータ binderhub_r_package(単一文字列)を binderhub_r_packages(リスト)にリネームし、デフォルト値を ['remotes', 'eegkit'] に変更
    • R (MRAN/CRAN) 追加ステップのコードを for pkg in binderhub_r_packages: ループに変更し、リスト内の各パッケージを順に登録するように修正
    • 対応する Markdown 見出し・期待結果を「各パッケージを登録する/登録したパッケージ名が R (MRAN/CRAN) に表示されること」に変更し、コード実装と整合させた
  • テスト手順-BinderHubアドオン-アドオン追加.ipynb
    • パラメータに binderhub_r_packages = ['remotes'] を追加
    • 「解析」タブクリックと「新しい解析環境を作成」クリックの間に、R (MRAN) +追加 ループステップを挿入し、解析環境作成前に remotes を登録するように修正

Ticket

Custom Test Configuration

  • RDM_REPOSITORY:
  • RDM_BRANCH:
  • RDM_MERGE:
  • OSF_IMAGE:
  • EMBER_IMAGE:
  • CAS_IMAGE:
  • MFR_IMAGE:
  • WB_IMAGE:
  • EXCLUDE_NOTEBOOKS:

@yacchin1205 yacchin1205 changed the title Listify binderhub_r_package and prepend remotes for install_github BinderHubアドオン repo2docker テストのRパッケージ指定をリスト化し remotes を追加 May 25, 2026
@yacchin1205 yacchin1205 changed the title BinderHubアドオン repo2docker テストのRパッケージ指定をリスト化し remotes を追加 [GRDM-59575 Workaround] BinderHubアドオン repo2docker テストに remotes を追加 May 25, 2026
@yacchin1205 yacchin1205 force-pushed the fix/binderhub-r-remotes branch from ec5e6ba to 9cc176a Compare May 25, 2026 23:48
@yacchin1205 yacchin1205 changed the title [GRDM-59575 Workaround] BinderHubアドオン repo2docker テストに remotes を追加 BinderHubアドオン E2EテストのR依存に remotes を追加 May 25, 2026
@yacchin1205 yacchin1205 changed the title BinderHubアドオン E2EテストのR依存に remotes を追加 [GRDM-59575 Workaround] BinderHubアドオン repo2docker テストに remotes を追加 May 25, 2026
@yacchin1205 yacchin1205 force-pushed the fix/binderhub-r-remotes branch from 9cc176a to 9016f79 Compare May 26, 2026 01:45
@yacchin1205 yacchin1205 merged commit fa7efe1 into RCOSDP:main May 26, 2026
13 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant