Skip to content

[GRDM-61282] Update CS-repo2docker to 2026.07.2 and fix figshare DOI handling#29

Draft
yacchin1205 wants to merge 2 commits into
RCOSDP:masterfrom
yacchin1205:feature/repo2docker-2026.07.2
Draft

[GRDM-61282] Update CS-repo2docker to 2026.07.2 and fix figshare DOI handling#29
yacchin1205 wants to merge 2 commits into
RCOSDP:masterfrom
yacchin1205:feature/repo2docker-2026.07.2

Conversation

@yacchin1205

@yacchin1205 yacchin1205 commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

CS-repo2dockerの参照を 2026.07.2 に更新します。

CS-binderhub 単体の動作には現行の参照のままでも問題はありませんが、CS-tljh-repo2docker の CI は「CS-binderhub と CS-repo2docker を同一環境に pip インストールし、binderhub ビルドバックエンド経由で Python パッケージ版の repo2docker が解析環境をビルドする」構成を検証しており、この使用場面では解析環境のビルドが失敗する問題への対処(RCOSDP/CS-repo2docker#30)を含む 2026.07.2 が必要です。

あわせて、figshare が新しい URL 形式を返す環境で DOI からのサービス起動・ビルドができない問題(GRDM-61282)への対応として、upstream の修正 jupyterhub/binderhub@6456ae7 を cherry-pick で取り込みます。CI の tests (v1.30, main) で失敗していた figshare provider テストはこの問題によるものです。

@codecov-commenter

codecov-commenter commented Jul 14, 2026

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 84.61538% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.32%. Comparing base (773c6fb) to head (d8327a6).
⚠️ Report is 31 commits behind head on master.

Files with missing lines Patch % Lines
binderhub/repoproviders.py 84.61% 2 Missing ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
+ Coverage   77.59%   78.32%   +0.72%     
==========================================
  Files          44       28      -16     
  Lines        3616     3336     -280     
  Branches       47        0      -47     
==========================================
- Hits         2806     2613     -193     
+ Misses        785      723      -62     
+ Partials       25        0      -25     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yacchin1205 yacchin1205 force-pushed the feature/repo2docker-2026.07.2 branch from 4634ca5 to 9c40da1 Compare July 14, 2026 07:00
rather than parse mutable URL schemes (since it has changed recently),
parse more stable doi scheme and use figshare API to resolve versions if needed

important fix: for dois without version, figshare resolves to latest, not 1
@yacchin1205 yacchin1205 changed the title Update CS-repo2docker to 2026.07.2 [GRDM-61282] Update CS-repo2docker to 2026.07.2 and fix figshare DOI handling Jul 14, 2026
@yacchin1205 yacchin1205 marked this pull request as draft July 14, 2026 08:25
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.

3 participants