Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cache Data and Remove Most Runtime Dependencies #24

Merged
merged 3 commits into from
Dec 19, 2023

Conversation

apainintheneck
Copy link
Owner

The goal here is to remove most runtime dependencies by caching all necessary country, region and currency data beforehand.

Follow-up to :

The goal here is to remove both runtime dependencies and slow
code that has be loaded. It's for performance and to just ship
less code which is always a good thing assuming things work the
same way.

I added two new cache generation commands one for lists and the
other for formatted output. This makes things much simpler at
runtime though of course all of the complexity is in pre-computing
and making sure that the pre-computed data is up-to-date.
@apainintheneck apainintheneck force-pushed the cache-data-and-remove-most-runtime-gems branch from 933ec73 to adb4fc6 Compare December 19, 2023 04:59
@apainintheneck
Copy link
Owner Author

The goal here is to remove both runtime dependencies and slow
code that has be loaded. It's for performance and to just ship
less code which is always a good thing assuming things work the
same way.

I added two new cache generation commands one for lists and the
other for formatted output. This makes things much simpler at
runtime though of course all of the complexity is in pre-computing
and making sure that the pre-computed data is up-to-date.

@apainintheneck apainintheneck changed the title WIP: Cache Data and Remove Most Runtime Dependencies Cache Data and Remove Most Runtime Dependencies Dec 19, 2023
@apainintheneck apainintheneck marked this pull request as ready for review December 19, 2023 05:17
cache/README.md Outdated Show resolved Hide resolved
@apainintheneck apainintheneck merged commit fcb4454 into main Dec 19, 2023
9 checks passed
@apainintheneck apainintheneck deleted the cache-data-and-remove-most-runtime-gems branch December 19, 2023 05:26
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