-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Unable to build latest
version from a tag
#10735
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
ha! Looks like we always expect latest to be a branch (even the option is called "Default branch"), but we aren't enforcing it readthedocs.org/readthedocs/projects/forms.py Lines 272 to 279 in 53e21bb
We are given users the option to select a tag! This was working previously because we were treating tags and branches kind of the same, but we aren't doing that anymore. Not sure about the fix here, probably more users were relying on us allowing latest to be a tag... |
I'll check tomorrow if this can be fixed, so our code accepts both, tags and branches as latest. |
We were kind of always expecting latest (machine created) to be a branch (the field is even named `default_branch`!). But we were allowing both branches and tags to be the default version (latest). https://github.com/readthedocs/readthedocs.org/blob/53e21bb3ee8a5fce0194eb5481fd81ac87d3d1fa/readthedocs/projects/forms.py#L272-L279 Closes #10735
* Versions: allow latest to be a tag We were kind of always expecting latest (machine created) to be a branch (the field is even named `default_branch`!). But we were allowing both branches and tags to be the default version (latest). https://github.com/readthedocs/readthedocs.org/blob/53e21bb3ee8a5fce0194eb5481fd81ac87d3d1fa/readthedocs/projects/forms.py#L272-L279 Closes #10735 * Update docstring * Create methods * Fix docstring
The fix was released today, latest can now point to a tag again. |
Details
Expected Result
Documentation is properly built, as it used to be the case.
I unfortunately haven't made a release in months, so I only noticed this recently. I could not find any git commits that could be linked to this apart from maybe this code, but this definitely used to work
Actual Result
Unable to build
latest
from a tag, as it fails at checkoutThe text was updated successfully, but these errors were encountered: