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
Fix #1522: fix 'str' object has no attribute 'attributes'
#1528
Fix #1522: fix 'str' object has no attribute 'attributes'
#1528
Conversation
Thanks for the fix here @leplatrem ! This was recently changed, in #1519, though it wasn't clear if there was going to be any side effects from the change. I guess there are side effects, which isn't too surprising. This is reverting to what we had before, so should definitely be safe. But I will first try to understand what is happening in #1522, or why the I'll want to aim to merge this in, reverting that change, before next week. I'm planning on a 2.0 final release on Monday or Tues. |
Excellent, thank you! 💚 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is revert to the old logic, it should be okay. If I don't come up with another fix or reason to keep the current logic, I'll look to merge this early next week for the 2.0 release. We can revisit the reverted change again in another release if we need.
This reverts commit b22a77f.
... and keep using the custom `link` HTML tag for CSS files included via the HTML theme option `extra_css_files`.
I updated this PR to take into consideration the review comments from @mitya57. It should work for all the Sphinx versions we support now. I'd be happy to merge this PR and release a new |
This looks good to me now. 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fix looks good to me, I'll get an RC prepared this afternoon so we can test the fix further.
I pushed up a change just standardizing the variable names, will wait on that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
2.0.0rc3 is now out with this fix. Thanks everyone! |
Fix #1522