google.cloud.datastore add_filter warnings

[dhimmel]

refs googleapis/python-datastore#504

To get rid of:

UserWarning: Detected filter using positional arguments. Prefer using the 'filter' keyword argument instead.

Untested locally, so do not merge unless CI passes and CI uses one of these filters.

[dhimmel]

3 CI tests failed due to

FAILED articat/tests/utils_test.py::test_download__empty - FileExistsError: [Errno 17] File exists: '/tmp/tmpsmeymxih'
FAILED articat/tests/utils_test.py::test_download__weird_valid_pattern - ValueError: Invalid pattern: '**' can only be an entire path component
FAILED articat/tests/utils_test.py::test_download__embedded_dir - FileExistsError: [Errno 17] File exists: '/tmp/tmp06uvrlun'

But this occurs on main as well.

[danielhfrank]

I'm running down the test failures that you mentioned, and believe it may be related to a newer version of fsspec being released since we last updated the repo. I'll continue working on that, and make sure that tests pass on both 3.10 and 3.11

[dhimmel]

Sounds good. I don't think there is any need for articat to support dependency versions lower than what we use internally in facets in case that simplifies the code updates.