Skip to content

update release automation scripts#857

Merged
houqp merged 2 commits intoapache:masterfrom
houqp:qp_release
Aug 12, 2021
Merged

update release automation scripts#857
houqp merged 2 commits intoapache:masterfrom
houqp:qp_release

Conversation

@houqp
Copy link
Member

@houqp houqp commented Aug 11, 2021

Which issue does this PR close?

related to #771

Rationale for this change

The current create-tarball script uses the version tag to extract the release commit. This could be error prone because it requires force git tag update on every subsequent release candidate update.

What changes are included in this PR?

  • Work with rc tag instead of final version during release preparation process so we don't need to do force git tag update on rc version bump.
  • Check for cargo publish in release verification automation process

Are there any user-facing changes?

no

@houqp houqp added the development-process Related to development process of DataFusion label Aug 11, 2021
@houqp houqp requested review from alamb and andygrove August 11, 2021 17:56
Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

Looks reasonable to me 👍

@houqp houqp merged commit 83ce64a into apache:master Aug 12, 2021
@houqp houqp deleted the qp_release branch August 12, 2021 06:30
unkloud pushed a commit to unkloud/datafusion that referenced this pull request Mar 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

development-process Related to development process of DataFusion

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants