Rename CACHE_DIRECTORY to .erb_lint_cache.#380
Conversation
From `.erb-lint-cache`. This is to align with the general renaming and standardization from `erb-lint` and `erblint` to `erb_lint`. Fixes Shopify#379.
etiennebarrie
left a comment
There was a problem hiding this comment.
Do you think we need to do anything about the migration or just document this in the release notes when releasing the new version?
|
Good question. Personally, I just rolled this into our app and it just had to rebuild the cache fresh the first time and I just had to update the I wouldn't consider this a breaking change or anything. Since we're not at v1 yet I think it's fair just to put it into the Release Notes / Let me know what you think and I can update the PR. |
Yeah I think it's good enough to have a good message. We don't have a changelog file but we use GitHub releases which show up in Dependabot PRs for example. Thanks for your contribution! |
|
👍 Awesome, thanks for the quick turnaround. Btw, I have another PR (#300) that is critical for anybody using the cache, which breaks it when there are configuration or Thanks again. |
|
Yes, I saw. I liked #373 quite a bit better, we just need some specs. |
|
Yep, I agree. That's much more clever. :) Thanks. |
From
.erb-lint-cache.This is to align with the general renaming and standardization from
erb-lintanderblinttoerb_lint.Fixes #379.