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

Caching docs in Deno.Kv and testable file system helpers #892

Draft
wants to merge 19 commits into
base: tm/push-docs-to-tags
Choose a base branch
from

Conversation

taras
Copy link
Member

@taras taras commented Feb 13, 2024

Motivation

I started rewriting the GitHub Actions workflows from #891 into Deno tasks. In the process, I realized that I would like to cache the contents of the docs directory so I can load it from Deno.Kv instead of untarballing the files in the Deno instance. As it was getting more complicated, I wanted to start building up helpers and ensure they were tested. I kept ballooning the requirements, but it's good practice.

Approach

  1. Created useCwd hook that uses CwdContext for storing the current context.
  2. Created useReadDir and useMkdir that are useCwd compatible
  3. Started writing test

TODO

  1. My useCwd doesn't actually work, it's something about the way I'm using scope. useReadDir and useMkdir don't actually get the set value.

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