Skip to content

feat: add bracket template cache#21

Merged
matthieugusmini merged 1 commit intomatthieugusmini:mainfrom
atrahy:feat/template-cache
May 4, 2025
Merged

feat: add bracket template cache#21
matthieugusmini merged 1 commit intomatthieugusmini:mainfrom
atrahy:feat/template-cache

Conversation

@atrahy
Copy link
Copy Markdown
Collaborator

@atrahy atrahy commented May 3, 2025

Add cache for bracket templates

Closes #17

  • Add cache package, the filesystem file modification time is used for invalidation
  • Split BracketTemplateLoader into two struct Client and Loader
  • Client is responsible for file fetch on github
  • Loader will get the file from either the cached file or fetch a new one

Copy link
Copy Markdown
Owner

@matthieugusmini matthieugusmini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome job @atrahy! Thanks a lot for your contribution 🎉

Thank You

@matthieugusmini matthieugusmini force-pushed the main branch 2 times, most recently from 2e5af05 to c59412c Compare May 4, 2025 08:26
@atrahy atrahy force-pushed the feat/template-cache branch 7 times, most recently from 63f7847 to 9ac6a5a Compare May 4, 2025 21:37
feat: swap file cache to KV db
feat: preprare log
feat: tidy go.mod
feat: open db in main
feat: generic cache Get & Set
feat: generic + facilitator
@atrahy atrahy force-pushed the feat/template-cache branch from 9ac6a5a to 2abc140 Compare May 4, 2025 22:27
Copy link
Copy Markdown
Owner

@matthieugusmini matthieugusmini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @atrahy! That is such a cool first contribution!

@matthieugusmini matthieugusmini merged commit 7f0c61e into matthieugusmini:main May 4, 2025
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.

Implement file based caching

2 participants