Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move legacy importer code out of Open Library #37

Closed
wants to merge 1 commit into from

Conversation

hornc
Copy link
Collaborator

@hornc hornc commented Feb 17, 2020

This moves the directory: openlibrary/catlog/
at SHA fab6433a1c6170e25e8faf0c6a8cefaf28155962
https://github.com/internetarchive/openlibrary/tree/fab6433a1c6170e25e8faf0c6a8cefaf28155962/openlibrary/catalog

out of the Open Library repo for archiving to assist with Python 3
upgrading, and mainting and extending the active import code.

Some code from catalog/ is in active use by OL, archiving this entire
directory will allow us to agressively trim back to the code we are
actually using in OL.

Part of Open Library issue internetarchive/openlibrary#3049

I have not included the test_data directories under marc/ https://github.com/internetarchive/openlibrary/tree/fab6433a1c6170e25e8faf0c6a8cefaf28155962/openlibrary/catalog/marc/tests/test_data and add_book/ https://github.com/internetarchive/openlibrary/tree/fab6433a1c6170e25e8faf0c6a8cefaf28155962/openlibrary/catalog/add_book/test_data because those are in active use on OL, and could be acquired from that repo if needed.

This moves the directory: openlibrary/catlog/
at SHA fab6433a1c6170e25e8faf0c6a8cefaf28155962
https://github.com/internetarchive/openlibrary/tree/fab6433a1c6170e25e8faf0c6a8cefaf28155962/openlibrary/catalog

out of the Open Library repo for archiving to assist with Python 3
upgrading, and mainting and extending the active import code.

Some code from catalog/ is in active use by OL, archiving this entire
directory will allow us to agressively trim back to the code we are
actually using in OL.
@hornc
Copy link
Collaborator Author

hornc commented Mar 24, 2021

Maybe this repo should be for active bots only. This code is purely archived. It will exist on this branch, and in the OpenLibrary repo history.

@hornc hornc closed this Mar 24, 2021
@cclauss cclauss deleted the ia-legacy-importer branch January 15, 2023 12:15
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.

1 participant