Skip to content

Another try for fixing ">" comparison for Octopress#2

Open
bdillahu wants to merge 1 commit intobelkadan:masterfrom
bdillahu:master
Open

Another try for fixing ">" comparison for Octopress#2
bdillahu wants to merge 1 commit intobelkadan:masterfrom
bdillahu:master

Conversation

@bdillahu
Copy link

Fixing usage with Octopress where ">" comparison doesn't work. Based on info from https://github.com/soonhokong pull request

I'm not a great Ruby person, so this may still have issues, but it seems like its been working for me.

You do have to be careful if you're changing layouts or something to actually touch the index files that are affected.

Fixing usage with Octopress where ">" comparison doesn't work. Based on info from https://github.com/soonhokong pull request
@belkadan
Copy link
Owner

I don't understand why > comparison doesn't just work when Time includes Comparable. I'm not actively focused on my jekyll fork right now, but I haven't even looked at the error on 1.9.2, and I'm guessing that this is not the problem.

@bdillahu
Copy link
Author

I'm afraid answering the "why > doesn't work" is beyond me at the moment. I understand the issue that this probably isn't the best fix, but wanted to offer it up in case it was some help.

Thanks for all your work on this fork!

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.

2 participants