You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by GitBox <gi...@apache.org> on 2020/07/24 13:40:50 UTC
[GitHub] [arrow-site] kszucs opened a new pull request #66: [Website] Add release note for 1.0.0
kszucs opened a new pull request #66:
URL: https://github.com/apache/arrow-site/pull/66
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-site] kou commented on a change in pull request #66: [Website] Add release note for 1.0.0
Posted by GitBox <gi...@apache.org>.
kou commented on a change in pull request #66:
URL: https://github.com/apache/arrow-site/pull/66#discussion_r460309619
##########
File path: _data/versions.yml
##########
@@ -16,16 +16,16 @@
# Database of the current version
#
current:
- number: '0.17.1'
- pinned_number: '0.17.*'
- date: '18 May 2020'
- git-tag: 'ff7ee06020949daf66ac05090753e1a17736d9fa'
- github-tag-link: 'https://github.com/apache/arrow/releases/tag/apache-arrow-0.17.1'
- release-notes: 'https://arrow.apache.org/release/0.17.1.html'
- mirrors: 'https://www.apache.org/dyn/closer.lua/arrow/arrow-0.17.1/'
- tarball_name: 'apache-arrow-0.17.1.tar.gz'
- mirrors-tar: 'https://www.apache.org/dyn/closer.lua/arrow/arrow-0.17.1/apache-arrow-0.17.1.tar.gz'
- java-artifacts: 'http://search.maven.org/#search%7Cga%7C1%7Cg%3A%22org.apache.arrow%22%20AND%20v%3A%220.17.1%22'
- asc: 'https://downloads.apache.org/arrow/arrow-0.17.1/apache-arrow-0.17.1.tar.gz.asc'
- sha256: 'https://downloads.apache.org/arrow/arrow-0.17.1/apache-arrow-0.17.1.tar.gz.sha256'
- sha512: 'https://downloads.apache.org/arrow/arrow-0.17.1/apache-arrow-0.17.1.tar.gz.sha512'
+ number: '1.0.0'
+ pinned_number: '1.0.*'
Review comment:
`'1.*'` may be better because we use Semantic Versioning since 1.0.0.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-site] kszucs commented on pull request #66: [Website] Add release note for 1.0.0
Posted by GitBox <gi...@apache.org>.
kszucs commented on pull request #66:
URL: https://github.com/apache/arrow-site/pull/66#issuecomment-663574462
Well, the other change seems a bit more invasive. If the changes are compatible I'd rather merge them separately.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-site] nealrichardson commented on pull request #66: [Website] Add release note for 1.0.0
Posted by GitBox <gi...@apache.org>.
nealrichardson commented on pull request #66:
URL: https://github.com/apache/arrow-site/pull/66#issuecomment-663624529
Well, whichever order we merge things in, I have merged this to my website branch and deployed it to https://enpiar.com/arrow-site/
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-site] nealrichardson commented on pull request #66: [Website] Add release note for 1.0.0
Posted by GitBox <gi...@apache.org>.
nealrichardson commented on pull request #66:
URL: https://github.com/apache/arrow-site/pull/66#issuecomment-663563492
How about I merge this into #63, add the finishing touches to the release blog post (i.e. filling in the number of commits and contributors) do a test deploy of that, and then merge to master assuming the test deploy looks as desired?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-site] nealrichardson merged pull request #66: [Website] Add release note for 1.0.0
Posted by GitBox <gi...@apache.org>.
nealrichardson merged pull request #66:
URL: https://github.com/apache/arrow-site/pull/66
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-site] kszucs edited a comment on pull request #66: [Website] Add release note for 1.0.0
Posted by GitBox <gi...@apache.org>.
kszucs edited a comment on pull request #66:
URL: https://github.com/apache/arrow-site/pull/66#issuecomment-663574462
Well, the other change seems a bit more invasive. If the patches are compatible I'd rather merge them separately.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org