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

adding a build environment dockerfile for jbrowse #1472

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

scottcain
Copy link
Member

Submitting a pull request

Please base your pull request based on the "dev" branch of JBrowse

The default branch is "master" but this contains the last stable release, and new pull requests should be based off of "dev"

Then set the base branch to "dev" on GitHub also

@scottcain scottcain requested a review from cmdcolin February 17, 2020 19:31
@scottcain
Copy link
Member Author

scottcain commented Feb 17, 2020

The image created from this dockerfile is intended to be a parent image for a JBrowse docker file, so that the author of a jbrowse docker file doesn't have to futz with doing this themselves. An example implementation of a JBrowse dockerfile will be forthcoming when this is available on docker hub.

(edited to change "base image" to "parent image"--I had the terminology wrong)

@scottcain
Copy link
Member Author

scottcain commented Feb 17, 2020

I nice side effect of having this docker file around is that it also serves as concise documentation of what is needed in a Debian-based environment for building JBrowse.

@cmdcolin
Copy link
Contributor

What happens after this PR is merged? It needs to be published still?

@scottcain
Copy link
Member Author

My understanding is also somewhat rudimentary, but what I think happens is that I have to build an image and then do a push of the image to docker hub. The reality is that this pull request doesn't really do much, other than provide documentation "describing" what we're putting on docker hub.

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