Skip to content

docs: more release docs improvements#3104

Merged
geruh merged 3 commits intoapache:mainfrom
kevinjqliu:kevinjqliu/more-release-docs
Mar 4, 2026
Merged

docs: more release docs improvements#3104
geruh merged 3 commits intoapache:mainfrom
kevinjqliu:kevinjqliu/more-release-docs

Conversation

@kevinjqliu
Copy link
Contributor

@kevinjqliu kevinjqliu commented Feb 26, 2026

Rationale for this change

Following #3030, more docs improvements

  • Add instruction to push the patch branch
  • Add checks that env vars are set
  • Upload to SVN without downloading all the artifacts first
Screenshot 2026-02-26 at 10 08 39 AM

Are these changes tested?

Yes, tested the svn import command locally

Are there any user-facing changes?

kevinjqliu and others added 3 commits February 26, 2026 10:01
Copy link
Member

@geruh geruh left a comment

Choose a reason for hiding this comment

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

Thanks Kevin, this is a good addition to guard against running with unset variables.

I tested a few with examples like this and looks good.

> export VERSION=0.8.0
export RC=1
export VERSION_WITH_RC=${VERSION}rc${RC}
export GIT_TAG=pyiceberg-${VERSION_WITH_RC}

> echo svn import "svn-release-candidate-${VERSION}rc${RC}" \
  "https://dist.apache.org/repos/dist/dev/iceberg/pyiceberg-${VERSION_WITH_RC}" \
  -m "PyIceberg ${VERSION_WITH_RC}"

svn import svn-release-candidate-0.8.0rc1 https://dist.apache.org/repos/dist/dev/iceberg/pyiceberg-0.8.0rc1 -m PyIceberg 0.8.0rc1

@geruh geruh merged commit 30d987f into apache:main Mar 4, 2026
5 checks passed
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.

2 participants