-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Branches and the "not using the most up to date version" note #1856
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
Comments
Agreed, we should probably remove this feature. It would be great if you could submit a PR working on this, the code is here: https://github.com/rtfd/readthedocs.org/blob/304de09d9253c464ea6ee5b8999d2d8886c83fdd/readthedocs/core/static-src/core/js/doc-embed/version-compare.js#L18 |
See also #1616. |
This is because |
Couldn't it just point to the default branch? |
Why does it try to parse a BRANCH name as version anyway? A release has a version and is a TAG, not a branch. |
@ThomasWaldmann currently RTD support both kind of work-flows (tags and branches), but if the project has at least one tag, those are given priority (see #3528) |
Maybe related to #3595 |
As commented #1856 (comment) and #1856 (comment). The issue wasn't a bug but related to an invalid version name (PEP440), which is documented on the docs https://docs.readthedocs.io/en/latest/versions.html#how-we-envision-versions-working. Also, the team has decided to implement #1616 |
In order to be able to add corrections to my docs, and FAQ entries and the like, I use release branches as the versions on RTD. For instance, I have a 0.6.x branch, then 0.7.x, 0.8.x.
This has worked fine for me in every way, except that it breaks the "You are not using the most up to date version of the library" feature.
My recommendations are the following:
Example message: http://reprozip.readthedocs.org/en/0.4/packing.html
The text was updated successfully, but these errors were encountered: