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

docs: fix incorrect version in documentation #586

Merged
merged 2 commits into from
Jun 19, 2024

Conversation

tigarmo
Copy link
Collaborator

@tigarmo tigarmo commented Jun 18, 2024

This PR contains two commits:

  • build(docs): fetch tags in readthedocs builds

Readthedocs builds don't fetch tags when cloning the repo. This is a problem
for us because Rockcraft's version comes from the tag (via setuptools-scm),
so do a fetch of the tags as a post-checkout step.

  • docs: use "dev" for non-release builds

The bug is this: If a build is done off of (say) 'main', the version coming
from 'git describe' will be something like 1.32.2.postXXXX. We previously
dropped the ".post*" suffix because it confused the spellchecker but that
just leaves the misleading "1.32.2" string. It's misleading because that's
an actual released version that does not contain a lot of the content
in 'main'.

So just use "dev" instead, to make it extra explicit that the docs refer
to an unreleased version of the software.

Fixes #584


For reference, this is the setting in RDT's configuration page that makes "stable" be the default version:
image

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jun 18, 2024
@tigarmo tigarmo force-pushed the work/CRAFT-3018-fix-docs-version branch from 29a326d to 1ef4b6d Compare June 18, 2024 14:30
tigarmo added 2 commits June 18, 2024 11:42
Readthedocs builds don't fetch tags when cloning the repo. This is a problem
for us because Rockcraft's version comes from the tag (via setuptools-scm),
so do a fetch of the tags as a post-checkout step.
The bug is this: If a build is done off of (say) 'main', the version coming
from 'git describe' will be something like 1.32.2.postXXXX. We previously
dropped the ".post*" suffix because it confused the spellchecker but that
just leaves the misleading "1.32.2" string. It's misleading because that's
an actual released version that does *not* contain a lot of the content
in 'main'.

So just use "dev" instead, to make it extra explicit that the docs refer
to an unreleased version of the software.
@tigarmo tigarmo force-pushed the work/CRAFT-3018-fix-docs-version branch from 1ef4b6d to cab5176 Compare June 18, 2024 14:45
@tigarmo tigarmo requested a review from a team June 18, 2024 15:56
@tigarmo tigarmo merged commit d5da187 into main Jun 19, 2024
15 checks passed
@tigarmo tigarmo deleted the work/CRAFT-3018-fix-docs-version branch June 19, 2024 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong Rockcraft version in readthedocs docs
3 participants