Skip to content
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

[GRDM-39566] JupyterLab 4.x対応 #4

Merged
merged 8 commits into from
Nov 15, 2023
Merged

Conversation

yacchin1205
Copy link
Collaborator

JupyterLab 4.xへの対応を行いました。このバージョンでは、以下の環境をサポートします。

  • JupyterLab 4.x
  • Jupyter Notebook 7.x
  • Jupyter Notebook 6.5.x (nbclassic)

以下の変更を行いました。

  • 設定ファイル等のJupyterLab 4.xへの対応
  • ファイルメニューへのSync to RDMボタン追加(Jupyter Notebook 7でもSync to RDMが利用できるように)

@yacchin1205 yacchin1205 changed the title [GRDM-39566] JupyterLab 4.0.x対応 [GRDM-39566] JupyterLab 4.x対応 Oct 10, 2023
@yacchin1205
Copy link
Collaborator Author

Check Releaseアクションの失敗は、jupyter-releaserのバグを踏んだのが原因のようです: jupyter-server/jupyter_releaser#529
jupyter-releaserの修正を検討中です。

@yacchin1205 yacchin1205 marked this pull request as ready for review October 11, 2023 00:56
@yacchin1205
Copy link
Collaborator Author

yacchin1205 commented Oct 11, 2023

Check Releaseアクションの失敗は、jupyter-releaserのバグを踏んだのが原因のようです: jupyter-server/jupyter_releaser#529 jupyter-releaserの修正を検討中です。

暫定措置として、jupyter_releaserをforkし、問題のコードを修正したものを使うようにしました。
https://github.com/RCOSDP/CS-jupyterlab-grdm/pull/4/files#diff-87a40ec3f710131fea2396404f334598e3e5a2e60e921de516d067cd23e3e401R25-R26

この問題は、すでにPRが出されていて、マージが検討されているようですので、時間が経てば本Workaroundは不要になるかと思います。 jupyter-server/jupyter_releaser#522

@ikfj ikfj merged commit b254d4f into RCOSDP:main Nov 15, 2023
3 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.

2 participants