Skip to content

Make Archive.get_entry_by_id public and deprecate _get_entry_by_id - #274

Open
Sriram-Nambiar wants to merge 1 commit into
openzim:mainfrom
Sriram-Nambiar:make-get-entry-by-id-public
Open

Sriram-Nambiar wants to merge 1 commit into
openzim:mainfrom
Sriram-Nambiar:make-get-entry-by-id-public

Conversation

@Sriram-Nambiar

Copy link
Copy Markdown

Fixes #225.

  • Exposes public Archive.get_entry_by_id(entry_id).
  • Deprecates Archive._get_entry_by_id(entry_id) with a DeprecationWarning pointing to get_entry_by_id.
  • Updates type hints in reader.pyi.
  • Updates tests in test_libzim_reader.py and asserts DeprecationWarning on _get_entry_by_id.
  • Updates CHANGELOG.md.

This branch has not been deployed

No deployments
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.

Remove private prefix of Archive._get_entry_by_id

1 participant