Skip to content
This repository has been archived by the owner on Jun 8, 2019. It is now read-only.

Add readme for yesod image usage #3

Merged
merged 1 commit into from
Jun 24, 2015
Merged

Add readme for yesod image usage #3

merged 1 commit into from
Jun 24, 2015

Conversation

tonyd256
Copy link
Contributor

No description provided.

RUN cabal install --dependencies-only -j4
```

You can build your container like this:
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we want to build the container here? I think that will install all the dependencies, and it looks like you'll have to install all the dependencies again when running docker-compose build.

Choose a reason for hiding this comment

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

I think the point here was that this is how to set it up without docker-compose. If you need database support, then you'd want to use docker-compose, and so should follow the next set of instructions. It's like the README version of a choose your own adventure. We should probably communicate this better though.

@tonyd256 tonyd256 force-pushed the td-yesod-readme branch 2 times, most recently from 975d811 to caa403f Compare June 24, 2015 17:26
@tonyd256
Copy link
Contributor Author

@jferris any comments for the readme in here? I think it looks good.

First, create a `Dockerfile` for your project. The one shown above is a good
starting place.

Test that your container builds before moving on:
Copy link
Contributor

Choose a reason for hiding this comment

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

If they test their container at this point and then switch to docker-compose, they'll need to reinstall the dependencies, which takes like 20 minutes. Is there a reason to test here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ya I think we can remove this stuff

@tonyd256
Copy link
Contributor Author

@jferris how does it look now?

@jferris
Copy link
Contributor

jferris commented Jun 24, 2015

Nice!

@tonyd256 tonyd256 merged commit 356d08f into master Jun 24, 2015
@tonyd256 tonyd256 deleted the td-yesod-readme branch June 24, 2015 20:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants