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

Port more 3.2.x code #5126

Merged
merged 14 commits into from Apr 1, 2016
Merged

Port more 3.2.x code #5126

merged 14 commits into from Apr 1, 2016

Conversation

yakky
Copy link
Member

@yakky yakky commented Apr 1, 2016

No description provided.

@yakky yakky added this to the 3.3 milestone Apr 1, 2016
@@ -30,7 +30,7 @@
# extensions coming with Sphinx (named 'sphinx.ext.*') or your custom ones.
#extensions = ['sphinx.ext.autodoc']

extensions = ['djangocms', 'sphinx.ext.intersphinx', 'sphinx.ext.todo']
extensions = ['djangocms', 'sphinx.ext.intersphinx']
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please don't do this, sphinx.ext.todo is actually useful
this is the correct fix for the sphinx issue 7979a06 and it's already in develop

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm backporting the fix in 3.2
I think that such test-related fixes should be first applied to earlier versions and then forwardported, or at least they should be advertised as needing for backporting, otherwise they get lost

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 88.696% when pulling 5620cd6 on yakky:merge/3.2.3.post1b into f10daf7 on divio:develop.

@yakky yakky merged commit 4005d20 into django-cms:develop Apr 1, 2016
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.

None yet

5 participants