Skip to content

By @slientrob, respect CNAME and README on compile#262

Closed
kennethormandy wants to merge 2 commits intosintaxi:masterfrom
kennethormandy:silentrob-prime
Closed

By @slientrob, respect CNAME and README on compile#262
kennethormandy wants to merge 2 commits intosintaxi:masterfrom
kennethormandy:silentrob-prime

Conversation

@kennethormandy
Copy link
Collaborator

By @slientrob, harp compile now respects CNAME and the README like it did for .git and .gitignore

@kennethormandy
Copy link
Collaborator Author

@silentrob I’ve added tests for this, but they aren’t passing. I tried compiling a couple of different ways. It seems like the README and the CNAME file still don’t make it through with this update.

@kennethormandy
Copy link
Collaborator Author

Sounds like #312 will help us do this in a more generally applicable way.

@kellydunn
Copy link

@kennethormandy , is there any reason not to merge #312 such that it will solve this issue? Currently, deploying harp as a static site with github pages requires me to manually add CNAME and my README.md back to the project after compilation.

@kennethormandy
Copy link
Collaborator Author

@kellydunn Hey! I would definitely like us merge it in, but it breaks many, many tests right now. I would also like to confirm that we’re not going to mess anything of with Browserify by doing this, but I would be willing to investigate that if someone else wants to tackle getting the tests to pass.

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