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

Release 0.4.4 #900

Closed
wants to merge 3 commits into from
Closed

Conversation

stephenfin
Copy link

@stephenfin stephenfin commented Apr 15, 2020

Do the necessary work to release 0.4.4 and get things working with Sphinx 3.x. After this is merged, I think someone will just need to to tag the relevant commit (HEAD~) and push the release to PyPI.

@ericholscher I assume this falls on you to do? /o\ Let me know if there's anything else I can do to help.

EDIT It might look like there's more going on here than is necessary, but I've broken things up with commits, as seen here.

I knew I shouldn't be doing this manually but it took me a while to
figure out what I had to do. I eventually followed the guide for
OpenStack Horizon [1]. Add a tox target to avoid this confusion in the
future.

[1] https://docs.openstack.org/horizon/latest/contributor/topics/translation.html

Signed-off-by: Stephen Finucane <stephen@that.guru>
Signed-off-by: Stephen Finucane <stephen@that.guru>

This is mostly a no-op though there is one new, thankfully simple
translation added and two removed.

Old, commented out translations are removed.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Signed-off-by: Stephen Finucane <stephen@that.guru>
setup.py Outdated Show resolved Hide resolved
@ericholscher
Copy link
Member

@agjohnson is the main point person on the theme.

* Fix definition lists looking different with Sphinx 2.0+
* Add compatibility with Sphinx 3.x
Copy link

Choose a reason for hiding this comment

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

Where was Sphinx 3.x compatibility added?

Copy link
Author

Choose a reason for hiding this comment

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

Commit a49a812

Copy link

Choose a reason for hiding this comment

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

3.0.0 was released a year later. That was for Sphinx 2.x compatibility:
sphinx-doc/sphinx#6088

Copy link

Choose a reason for hiding this comment

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

Sphinx 2.x compatibility is still ongoing:
https://github.com/readthedocs/sphinx_rtd_theme/projects/1

Copy link
Contributor

Choose a reason for hiding this comment

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

"Improve compatibility with Sphinx 2.x" is more appropriate

@stephenfin
Copy link
Author

I wonder if I should roll in dropping support for Python 2.7 and 3.4 here? I'm thinking it would be better get this out the door and cut a 1.0.0 release that does that at a later date, but I'm willing to be overruled 😄

@stephenfin
Copy link
Author

Okay, it seems this is clearly not ready so it's probably best I close it. Feel free to cherry-pick any of the commits here that might be useful.

@stephenfin stephenfin closed this Apr 18, 2020
@utzig
Copy link

utzig commented May 13, 2020

Why was this abandoned?

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