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

Copy Root README to dist #85

Merged
merged 10 commits into from
May 14, 2017
Merged

Conversation

dbfannin
Copy link
Contributor

@dbfannin dbfannin commented May 8, 2017

This PR copies the README.MD file at the root of the generated project to the /dist directory.

The thought is that the user shouldn't have to maintain multiple README files. However, if you see value in there being a separate README for distributable files, let me know and I can modify.

@jdjuan
Copy link
Collaborator

jdjuan commented May 8, 2017

This looks good to me. I will test it before tomorrow in case everybody else is busy. Thanks @dbfannin ! 🙂

@jvandemo
Copy link
Owner

jvandemo commented May 9, 2017

@dbfannin — Looks great, thanks!

There should be no need for the / in path.join(rootFolder, '/README.MD').

Can you please update it to path.join(rootFolder, 'README.MD')?

Thanks again!

davidf added 5 commits May 9, 2017 07:34
@dbfannin
Copy link
Contributor Author

dbfannin commented May 9, 2017

Resolved conlicts and updated the path.join. @jvandemo, this should be good to go now. Thanks!

@jvandemo
Copy link
Owner

@dbfannin — Thank you for the rebase.

I think the copy:readme task is defined twice in gulpfile.js:

Can you fix that please?

Thanks again, David!

@dbfannin
Copy link
Contributor Author

Well that's embarrassing. Not sure how I managed to do that. It should be good to go now (fingers crossed)

@jdjuan
Copy link
Collaborator

jdjuan commented May 10, 2017

It looks good to me now 👍

@jvandemo jvandemo merged commit 2cdb7c7 into jvandemo:master May 14, 2017
@jvandemo
Copy link
Owner

Merged and released as v10.1.0.

@dbfannin — Thank you for your contribution! 👍
@jdjuan — Thank you for your review 👍

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