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

fix: align tmp and cache dir #4257

merged 1 commit into from Dec 28, 2022

fix: align tmp and cache dir #4257

merged 1 commit into from Dec 28, 2022


Copy link

@PeterSchafer PeterSchafer commented Dec 22, 2022

What does this PR do?

Avoid file system conflicts on linux and align where the CLI stores files on the local system.

On linux, temp dir resolves to /tmp and we extended it to /tmp/snyk. If any user chose to use create a folder or file with the same name, there is a conflict.

@PeterSchafer PeterSchafer requested a review from a team as a code owner December 22, 2022 17:07
@PeterSchafer PeterSchafer force-pushed the fix/HMMR-626 branch 5 times, most recently from e72e1d1 to 6abf792 Compare December 27, 2022 11:16
@PeterSchafer PeterSchafer changed the title fix: make temp dir unique by adding a timestamp fix: align tmp and cache dir Dec 27, 2022
Copy link
Contributor Author

Pair Reviewed with @asaf92 after doing additional changes.

@PeterSchafer PeterSchafer merged commit 13fea9e into master Dec 28, 2022
12 checks passed
@PeterSchafer PeterSchafer deleted the fix/HMMR-626 branch December 28, 2022 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

None yet

2 participants