Skip to content

refactor(data)!: move JSON files to ./data/cache/v0#30

Merged
bassosimone merged 12 commits intomainfrom
refactor/cache
Nov 26, 2025
Merged

refactor(data)!: move JSON files to ./data/cache/v0#30
bassosimone merged 12 commits intomainfrom
refactor/cache

Conversation

@bassosimone
Copy link
Collaborator

@bassosimone bassosimone commented Nov 26, 2025

This refactor addresses friction I felt while I was working on #28.

Having the data at the same level of the scripts used to generate them significantly makes it harder to inspect what changed.

Additionally, scoping to a specific data version allows us to keep the current files around for as long as we need them, but allows for incrementally extending the data fetching pipeline.

BREAKING CHANGE: renamed IQBCache cache_dir parameter to data_dir and let the cache logic handle and know which versions of the cache are available and what to actually do with them.

This refactor addresses friction I felt while I was working on
#28.

Having the data at the same level of the scripts used to generate
them significantly makes it harder to inspect what changed.

Additionally, scoping to a specific data version allows us to
keep the current files around for as long as we need them, but
allows incrementally extending the data fetching pipeline.
@bassosimone bassosimone changed the title refactor(data): move JSON files to ./data/cache/v0 refactor(data)!: move JSON files to ./data/cache/v0 Nov 26, 2025
@bassosimone bassosimone merged commit b4b361f into main Nov 26, 2025
5 checks passed
@bassosimone bassosimone deleted the refactor/cache branch November 26, 2025 09:42
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.

1 participant