You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/05/08 03:36:07 UTC
[GitHub] [arrow-datafusion] jychen7 opened a new pull request, #2484: #2392 release script
jychen7 opened a new pull request, #2484:
URL: https://github.com/apache/arrow-datafusion/pull/2484
# Which issue does this PR close?
could help #2392
# Rationale for this change
generalize release script by adding arguments
# What changes are included in this PR?
<!--
There is no need to duplicate the description in the issue here but it is sometimes worth providing a summary of the individual changes in this PR.
-->
# Are there any user-facing changes?
<!--
If there are user-facing changes then we may require documentation to be updated before approving the PR.
-->
<!--
If there are any breaking changes to public APIs, please add the `api change` label.
-->
# Notes
was used in https://github.com/apache/arrow-datafusion/pull/2187
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-datafusion] andygrove merged pull request #2484: MINOR: Parameterize changelog script
Posted by GitBox <gi...@apache.org>.
andygrove merged PR #2484:
URL: https://github.com/apache/arrow-datafusion/pull/2484
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-datafusion] andygrove commented on a diff in pull request #2484: #2392 changelog script
Posted by GitBox <gi...@apache.org>.
andygrove commented on code in PR #2484:
URL: https://github.com/apache/arrow-datafusion/pull/2484#discussion_r867506738
##########
dev/release/README.md:
##########
@@ -113,9 +113,7 @@ git commit -a -m 'Update version'
### Update CHANGELOG.md
-Manually edit the base version tag argument in
-`dev/release/update_change_log-{ballista,datafusion}.sh`. Commits
-between the base verstion tag and the latest upstream master will be used to
+Define release branch (e.g. `master`), base version tag (e.g. `7.0.0`) and future version tag (e.g. `8.0.0`). Commits between the base verstion tag and the release branch will be used to
Review Comment:
```suggestion
Define release branch (e.g. `master`), base version tag (e.g. `7.0.0`) and future version tag (e.g. `8.0.0`). Commits between the base version tag and the release branch will be used to
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-datafusion] jychen7 commented on pull request #2484: #2392 changelog script
Posted by GitBox <gi...@apache.org>.
jychen7 commented on PR #2484:
URL: https://github.com/apache/arrow-datafusion/pull/2484#issuecomment-1120343132
@andygrove I hope it could help release of 8.0.0 or future release
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org