jfmengels/elm-review
1
1
Fork 0
mirror of https://github.com/jfmengels/elm-review.git synced 2024-11-23 23:05:35 +03:00
Code Issues Packages Projects Releases Wiki Activity

Write down some thoughts on caching

Browse Source
This commit is contained in:
Jeroen Engels 2019-11-19 23:39:53 +01:00
parent 08df815a0f
commit f8632c20bc
1 changed files with 13 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
documentation/design/multi-file.md
View File

@ -55,6 +55,19 @@ Here are a few examples:
- Load dependencies before loading the files (wrt the `ProcessContext`) - Load dependencies before loading the files (wrt the `ProcessContext`)
- Make `Project` handle the list of files, the parsing, the graph creation, etc. - Make `Project` handle the list of files, the parsing, the graph creation, etc.
## Caching
- For multi-rules
- Add a way to decode/encode the context for each file
- If the rule supports encoding/decoding, in `Review.Test` check that `context == decode <| encode context` for every file
- Store the contexts for each file/rule to the file system
- Load the contexts for each file/rule from the file system
- Figure out what to do if the initial context changes (discard the cache?)
- Propose a way in the CLI to
- remove the cache?
- run without cache?
- specify the cache location?
## Work on errors ## Work on errors
- [X] Define a way to report errors in other files? - [X] Define a way to report errors in other files?