Skip to content

ci: fix RTD builds when last git tag is more than 50 commits behind #1001

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

Merged
merged 2 commits into from
Feb 27, 2025

Conversation

agriyakhetarpal
Copy link
Contributor

@agriyakhetarpal agriyakhetarpal commented Feb 27, 2025

Description

At the time of writing, the "latest" version of the scikit-build-core documentation website (https://scikit-build-core.readthedocs.io/en/latest/) shows documentation pages from an ancient version. This pull request should fix that.

Additional context

https://docs.readthedocs.com/platform/stable/build-customization.html#unshallow-git-clone

@LecrisUT
Copy link
Collaborator

Ah, this is a fun situation, we just crossed the 50 commit distance so it didn't got the last tag. I was wondering why the git distance didn't show a much bigger number

@LecrisUT LecrisUT changed the title Fix outdated documentation builds on the "latest" version fix: RTD documentation build when last git tag is more than 50 commits behind Feb 27, 2025
@LecrisUT LecrisUT changed the title fix: RTD documentation build when last git tag is more than 50 commits behind fix: RTD builds when last git tag is more than 50 commits behind Feb 27, 2025
@henryiii henryiii merged commit 24d4803 into scikit-build:main Feb 27, 2025
60 checks passed
@henryiii henryiii changed the title fix: RTD builds when last git tag is more than 50 commits behind ci: fix RTD builds when last git tag is more than 50 commits behind Feb 27, 2025
@agriyakhetarpal agriyakhetarpal deleted the fix/latest-docs branch February 27, 2025 14:59
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.

3 participants