Skip to content
This repository was archived by the owner on Jul 27, 2022. It is now read-only.

docs: remove code, and redirect to apache arrow site#52

Merged
pauldix merged 1 commit intomainfrom
alamb/redirect_to_new_home
Jul 26, 2022
Merged

docs: remove code, and redirect to apache arrow site#52
pauldix merged 1 commit intomainfrom
alamb/redirect_to_new_home

Conversation

@alamb
Copy link
Contributor

@alamb alamb commented Jul 23, 2022

Re #41 and apache/arrow-rs#2030

Rationale

We have donated object_store to the ASF and so we should remove this copy to make sure active development continues int he correct place

Changes

Remove all code and update readme to point at new location

@wjones127
Copy link
Contributor

This seems good, though I would have thought it would be sufficient to just add a message in the README and archive the GitHub repo (which will show a banner in the GitHub UI).

@alamb
Copy link
Contributor Author

alamb commented Jul 25, 2022

This seems good, though I would have thought it would be sufficient to just add a message in the README and archive the GitHub repo (which will show a banner in the GitHub UI).

I am happy to do that as well -- I don't have a strong preference. @tustvold any thoughts?

@tustvold
Copy link
Contributor

I think just updating the README and archiving the repo should be good enough, it will also make it easier to refer to the code "as it was" in case of any issues

@alamb alamb force-pushed the alamb/redirect_to_new_home branch from ab4af8d to d8d0425 Compare July 25, 2022 13:50
@alamb alamb changed the title remove code, and redirect to apache arrow site doc: remove code, and redirect to apache arrow site Jul 25, 2022
@alamb alamb changed the title doc: remove code, and redirect to apache arrow site docs: remove code, and redirect to apache arrow site Jul 25, 2022
@alamb alamb force-pushed the alamb/redirect_to_new_home branch from a0a47e9 to d4b7833 Compare July 25, 2022 13:58
@alamb alamb requested a review from tustvold July 26, 2022 21:48
@pauldix pauldix merged commit 9fd1884 into main Jul 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants