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

Add support for Jekyll 1.0.0 #7

Closed
wants to merge 6 commits into from
Closed

Add support for Jekyll 1.0.0 #7

wants to merge 6 commits into from

Conversation

cgarvis
Copy link

@cgarvis cgarvis commented Apr 30, 2013

No description provided.

@cgarvis
Copy link
Author

cgarvis commented May 1, 2013

@mattmanning I'm having some issues getting Jekyll to build on heroku. I'm getting the following error message

ERROR: YOUR SITE COULD NOT BE BUILT:
              ------------------------------------
              Post 0000-00-00-welcome-to-jekyll.markdown.erb does not have a valid date.

That file is a test file. It looks like Jekyll is building on top of itself. Does heroku install the gems in the same path as the app?

@brandonweiss
Copy link

I had this same problem. You need to exclude the vendor directory in your configuration. Like so: exclude: ["vendor"].

@cgarvis
Copy link
Author

cgarvis commented May 10, 2013

I've since moved to doing the jekyll build using the asset compiler. If I where to finish this out, I would create a config for this that could then be merged with project's jekyll config. I believe Jekyll 1.0 now supports this.

@brandonweiss
Copy link

Oh, interesting, so you've started doing all the building and whatnot via rake assets:precompile?

@oddsve
Copy link

oddsve commented May 24, 2013

👍

1 similar comment
@parkr
Copy link

parkr commented May 29, 2013

👍

@ryanmasondavies
Copy link

Would be great to see this merged 👍

@cgarvis
Copy link
Author

cgarvis commented Aug 20, 2013

You can see how I implemented it on my blog's rakefile:

https://github.com/cgarvis/chrisgarvis.com/blob/master/Rakefile#L5

@cgarvis cgarvis closed this Aug 20, 2013
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.

5 participants