Add default timeout to cache memoize

[KarimTayie]

[peterbe]

Can't you just leave it instead? I.e. if you do:

>>> from django.core.cache import cache
>>> cache.set('foo', 'bar')

it will apply whatever's in django.conf.settings.CACHES["default"].

[peterbe]

Oh I see. I think you can just use:

>>> from django.core.cache import cache
>>> cache.set('foo', 'bar', timeout=object())

is the exact same as not specifying it.

[peterbe]

I can't merge this because the CI tests aren't running. Any idea why?

[KarimTayie]

Hello, sorry for taking so long to reply.

I can't merge this because the CI tests aren't running. Any idea why?

I can't find the CI tests, can you please show me the logs?

[peterbe]

Let's see if this restarts the CI tests.

[peterbe]

Can you merge in the latest upstream master into your branch and push. Perhaps that'll pick up the new GitHub Actions checks.

[KarimTayie]

From github.com:peterbe/django-cache-memoize
 * branch            master     -> FETCH_HEAD
Already up to date.

already pulled the latest upstream master into this pr. from this screenshot, all looks good.

[peterbe]

Let's try if close and re-opening triggers the CI tests.

[peterbe]

What the heck?! I don't know why the tests aren't running. I'll have to take a look some day when I have time.

[YPCrumble]

@KarimTayie @peterbe my update in #48 should make the tests run via https://github.com/peterbe/django-cache-memoize/pull/48/commits/8470d5ba6feaf10519a5bd2ca360187fde726d7e

[peterbe]

@KarimTayie can you merge in master into your branch and push that that should hopefully fix your tests.