Skip to content

Conversation

@jorenham
Copy link
Contributor

@jorenham jorenham commented Dec 5, 2024

Because according to docs:

Because it never needs to evict old values, this is smaller and faster than lru_cache() with a size limit.

On python < 3.9 it will keep on using functools.lru_cache

@jorenham
Copy link
Contributor Author

jorenham commented Dec 5, 2024

That github code editor isn't the smartest eh?

@jorenham
Copy link
Contributor Author

jorenham commented Dec 5, 2024

it should be fixed now

@jorenham
Copy link
Contributor Author

jorenham commented Dec 6, 2024

I believe the CI failure is the same codecov issue that we ignored in #5 @csala

@csala
Copy link
Owner

csala commented Dec 18, 2024

I believe the CI failure is the same codecov issue that we ignored in #5 @csala

It is. Merging this now.

@csala csala merged commit 0deb95d into csala:master Dec 18, 2024
6 of 13 checks passed
@jorenham jorenham deleted the functools.cache branch December 18, 2024 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants