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

♻ replace env with cache #9

Open
hgwood opened this issue Feb 21, 2022 · 0 comments
Open

♻ replace env with cache #9

hgwood opened this issue Feb 21, 2022 · 0 comments
Assignees

Comments

@hgwood
Copy link
Owner

hgwood commented Feb 21, 2022

Currently the code in env.js loads a bunch of stuff from the .petibrugnon directory and makes it available to all other code files. The env.js code makes assumption about the state of the file system and the rest of the code files make assumptions about what they'll find in what they import from env.js, which might be wrong. All of this makes the program brittle. Note that most of what can change in env is stuff downloaded from the CodeJam API.

I should replace the env mechanism by caching. The program should simply fetch what it needs from the CodeJam API and cache it in local files. Only paths should come from env. I should use the same technique used for credentials (see here), but with different cache rules.

@hgwood hgwood self-assigned this Feb 21, 2022
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

No branches or pull requests

1 participant