Skip to content

Conversation

@rhshadrach
Copy link
Member

@rhshadrach rhshadrach commented Feb 3, 2026

We should only set the pandas version when the ref matches a valid SemVer string.

@rhshadrach rhshadrach requested a review from mroeschke as a code owner February 3, 2026 22:25
@rhshadrach
Copy link
Member Author

/preview

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2026

Website preview of this PR available at: https://pandas.pydata.org/preview/pandas-dev/pandas/64013/

@rhshadrach
Copy link
Member Author

I'm not familiar with how the banner works, I'm still not seeing the dev banner here.

if [ -n "${{ github.event.inputs.version }}" ]; then
echo "PANDAS_VERSION=${{ github.event.inputs.version }}" >> "$GITHUB_ENV"
else
elif [[ "${GITHUB_REF_NAME:1}" =~ ^[0-9]+\.[0-9]+\.[0-9]+$ ]]; then
Copy link
Member

Choose a reason for hiding this comment

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

Will that allow rc versions? (those get currently uploaded)

i.e. anytime that the if check passes in the "Upload prod docs" step, we should have a value set for PANDAS_VERSION (not fully sure if it would otherwise error, or fill in an empty string and upload the docs to the wrong directory)

@jorisvandenbossche
Copy link
Member

I'm not familiar with how the banner works, I'm still not seeing the dev banner here.

I forgot to answer about that on the issue, but that is unrelated, and just explicitly disabled in our conf.py

@jorisvandenbossche jorisvandenbossche added this to the 3.0.1 milestone Feb 4, 2026
@jorisvandenbossche jorisvandenbossche added Docs CI Continuous Integration labels Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous Integration Docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DOC: dev docs missing version

2 participants