lukasschwab / arxiv.py

Python wrapper for the arXiv API

MIT License

1.07k stars 120 forks source link

Improve Result and subclass methods for Python builtins #64

Closed lukasschwab closed 3 years ago

lukasschwab commented 3 years ago

Description

Note: this repo's releases have been frequent enough that I'll let this PR sit until Friday/Saturday.

Updates __repr__ implementations to format constructor arguments using repr(...). Before this change, quotes were missing from what should have been string literals in the __repr__ return values: arxiv.Result.Author(Some Name) should be arxiv.Result.Author("Some Name").
Adds __eq__ helpers for checking equality; relies naively on (Result).entry_id, (Result.Author).name, and (Result.Link).href.

Breaking changes

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

Minor breaking changes. I'm tempted to call this release 1.1.1, but changing equality behavior probably merits 1.2.0.

Changes equality checks: behavior before this change is to compare object addresses.
Changes __repr__, but that's a bug fix: nothing should programmatically depend on repr().

Relevant issues

List GitHub issues relevant to this change.

Fix #63.

Checklist

[x] All documentation is regenerated: run make docs.
[x] (If appropriate) README.md example usage has been updated.