-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Python 3.4 processing of content results in CRITICAL: unorderable types: Article() < Article() #483
Comments
Hi salvor7, Seems like this "just worked" with Python 2 but with Python 3 is throwing an error. A simple fix, but this affecting other themes and I would like to fix all at the same time - I could do this perhaps tomorrow but would need the PR to be accepted, etc. In the interim, if you want this to work, in zurb-f5-basic/tags.html change: Also, I've tested both ways working in Python 2 but not Python 3. But I Thanks, |
I've been using Python 3 since we'll before I started using pelican.
…On Apr 16, 2017 2:10 PM, "Josh Morel" ***@***.***> wrote:
Hi salvor7,
Seems like this "just worked" with Python 2 but with Python 3 is throwing
an error. A simple fix, but this affecting other themes and I would like to
fix all at the same time - I could do this perhaps tomorrow but would need
the PR to be accepted, etc.
In the interim, if you want this to work, in zurb-f5-basic/tags.html
change:
{% for article in articles|sort %}
to
{% for article in articles %}
Also, I've tested both ways working in Python 2 but not Python 3. But I
would like to confirm, did you just upgrade to using Python 3?
Thanks,
Josh
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#483 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AE5PMCMLwgyeEgtW-wnhgycQ1zG5DiC4ks5rwoPNgaJpZM4LwobW>
.
|
Hi Andrew, I put in a PR - which should fix it #506 So was this theme working before for you or did you just start trying it? Maybe it was a combination of Python 3 and the latest Pelican breaking that piece of code? Either way, once the PR is merged it will work with both Python 2 and 3. FYI - I had edited out that part of my comment because I thought it was extraneous - but since you've replied to it I added it back in. |
Thanks Josh
Andrew Brown
…On Wed, Apr 19, 2017 at 3:59 AM, Josh Morel ***@***.***> wrote:
Hi Andrew, I put in a PR - which should fix it #506
<#506>
So was this theme working before for you or did you just start trying it?
Maybe it was a combination of Python 3 and the latest Pelican breaking that
piece of code? Either way, once the PR is merged it will work with both
Python 2 and 3.
FYI - I had edited out that part of my comment because I thought it was
extraneous - but since you've replied to it I added it back in.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#483 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AE5PMHlDYPJnBcLq-QtvV42LAyi3avaWks5rxekHgaJpZM4LwobW>
.
|
It doesn't look like this issue made it over here from the pelican repo.
getpelican/pelican#1591
I just experience this error with zurb-F5-basic myself.
Sorry if this was raised under another name that I'm not seeing for what it is. My searches of the repo and closed tickets didn't turn up anything like it.
The text was updated successfully, but these errors were encountered: