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

Use --label to set labels #48

Merged
merged 1 commit into from
Dec 31, 2021
Merged

Use --label to set labels #48

merged 1 commit into from
Dec 31, 2021

Conversation

TimoRoth
Copy link
Contributor

This depends on jupyterhub/repo2docker#1097 being merged.

We'd like to use images which are built from custom Dockerfiles, for which --appendix does not work.
The aforementioned PR adds a more generic --label option, and this makes use of it.

@jtpio
Copy link
Member

jtpio commented Nov 29, 2021

Thanks @TimoRoth for this and the upstream PR!

a more generic --label option

This sounds like a more robust way 👍

@jtpio jtpio added the enhancement New feature or request label Nov 29, 2021
@jtpio jtpio mentioned this pull request Dec 2, 2021
@jtpio
Copy link
Member

jtpio commented Dec 16, 2021

@TimoRoth looks like jupyterhub/repo2docker#1097 has been merged.

Mind rebasing the PR to grab the latest changes? Which might help fix the CI. Thanks!

Copy link
Member

@jtpio jtpio left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@jtpio jtpio merged commit fe7d689 into plasmabio:master Dec 31, 2021
@jtpio
Copy link
Member

jtpio commented Dec 31, 2021

For those who want to use this feature, don't forget to pull the latest repo2docker image with:

docker pull quay.io/jupyterhub/repo2docker:main

Thanks again @TimoRoth!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants