Deprecate `(Search|Client).get` in favor of `(Search|Client).results`

[lukasschwab]

Description

Makes this package's API more literate.

It's not obvious what it means to "get" a Search; Results are the true object for the verb "get", but they're absent from the method call construction.

results seems more like an accessor (accessing the results on the Search, or accessing results in the Client that correspond to a Search).

• Duplicates the existing *.get(...) logic in *.results(...) methods.
• Converts the *.get(...) methods into thin wrappers of the corresponding *.results(...) methods.
• Updates README examples and tests to use *.results(...).

Still considering: client.run(search) or client.getResults(search) or something. Will merge after I reach some conclusion.

Breaking changes

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

None; the old *.get() methods remain supported (as simple wrappers of *.results().

These deprecated methods should be removed at the point that the results signatures need to change. This is unlikely to happen:

• One method is niladic; the other takes a Search object, the internals of which can change without this method signature changing.
• This project resists breaking changes to Search construction and methods that fetch results because they're it's primary interface.

Relevant issues

List GitHub issues relevant to this change.

N/A

Checklist

• [x] (If appropriate) README.md example usage has been updated.
• [x] Conclude about client method name.
• [ ] Increment the version number to 1.3.0.

[lukasschwab]

Should use warnings to warn of deprecation

Edit: added in 1726660 ✅

[lukasschwab]

Should only merge this when I'm really ready to roll the release to prevent publishing ~1.2.1~ 1.3.0 docs before *.results(...) is available.