lukasschwab / arxiv.py

Python wrapper for the arXiv API

MIT License

1.11k stars 123 forks source link

Separate HTTP request from `feedparser.parse` #136

Closed lukasschwab closed 1 year ago

lukasschwab commented 1 year ago

Description

Separates HTTP request from feedparser.parse call.
Sets a package-specific default User-Agent header.
[x] TODO: add requests to requirements.txt.
[ ] ~TODO: support a user-agent override (or all-headers override) for the client.~ Leaving this for a future PR to minimize the documentation updates.

Breaking changes

List any changes that break the API usage supported on master.

Breaking change; probably necessitates a 2.0.0 release.

Breaks exception types.
Breaks internal _parse_feed and __try_parse_feed function signatures.

Relevant issues

List GitHub issues relevant to this change.

Part of stability initiatives per #129.

Checklist

[ ] ~(If appropriate) README.md example usage has been updated.~

lukasschwab commented 1 year ago

Open question: should v2 eliminate the arxiv.Search(...).results method to encourage client reuse?

Edit: it'll deprecate but not eliminate; see #137.