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

anything beyond the most basic introduction should be in the documentation to avoid redundancy #61

Merged
merged 2 commits into from
Mar 29, 2019

Conversation

dbu
Copy link
Contributor

@dbu dbu commented Mar 11, 2019

fix #47

README.md Outdated Show resolved Hide resolved
@dbu
Copy link
Contributor Author

dbu commented Mar 11, 2019

i added some of the things i remove here in sculpin/sculpin.io#119

@bellisk
Copy link
Contributor

bellisk commented Mar 11, 2019

@dbu I noticed that in #63 I added more information to the section that you'd already stripped out, oops. Let's look at it quickly together and see how best to arrange it?

@dbu
Copy link
Contributor Author

dbu commented Mar 21, 2019

@beryllium i rebased now that the webpack encore thing is merged, and added setup instructions for composer and yarn so its really a complete instructions. i also tested the steps and got a running site 🎉

README.md Outdated Show resolved Hide resolved
@beryllium beryllium merged commit ba3b23f into sculpin:master Mar 29, 2019
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.

Make skeleton installable with composer create-project
3 participants