Skip to content

Add improved support for git and GitHub within JupyterLab #231

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jmunroe
Copy link
Contributor

@jmunroe jmunroe commented Jul 25, 2025

To support git (using the JupyterLab git extension) and GitHub (through gh-scoped-creds), I propose using the postBuild feature of repo2docker to add these additional packages. This means that the environment.yml file remains untouched while adding in these two-packages as 'standard' cookbooks.

Copy link

github-actions bot commented Jul 25, 2025

👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below.
🔍 Git commit SHA: 8c07ce4
✅ Deployment Preview URL: https://ProjectPythia.github.io/cookbook-template/_preview/231

set -ex

# Add support for developing with git and GitHub
conda install -c conda-forge jupyterlab-git gh-scoped-creds
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is postBuild run from within the conda environment? Or does this need to specify -n notebook?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My understanding it is run within the already activated conda environment.

@jukent
Copy link
Contributor

jukent commented Jul 31, 2025

Is the new plan to close this?

@jmunroe
Copy link
Contributor Author

jmunroe commented Jul 31, 2025

Could we leave this open for the moment? I am currently working if this is still needed or not (and in discussion internally with 2i2c).

I'll swap it to DRAFT for now.

@jmunroe jmunroe marked this pull request as draft July 31, 2025 19:49
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