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

Rewrite the cache store to match that done on impsort #833

Closed
hazendaz opened this issue Mar 6, 2024 · 1 comment · Fixed by #837
Closed

Rewrite the cache store to match that done on impsort #833

hazendaz opened this issue Mar 6, 2024 · 1 comment · Fixed by #837
Assignees

Comments

@hazendaz
Copy link
Member

hazendaz commented Mar 6, 2024

Currently as discovered in a port attempt over to impsort, the cache was written, then closed, reopened, sorted, and removed the timestamp incorrectly after sorting instead of before. We need to do same here. See revelc/impsort-maven-plugin#90

@hazendaz hazendaz self-assigned this Mar 6, 2024
@hazendaz
Copy link
Member Author

hazendaz commented Mar 9, 2024

Work is done and will be send in next day or two.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant