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

feat: Delete local .iac-data directory after scan is finished, using fs [CC-765] #1769

Closed
wants to merge 1 commit into from

Conversation

ipapast
Copy link
Contributor

@ipapast ipapast commented Mar 25, 2021

What does this PR do?

In a previous task, we introduced the download and extraction of files in a local cache directory, called '.iac-data'.

We do not need the files anymore after the scan/test, so we decided to clean up that local directory after the tests finishes.

How should this be manually tested?

  • Run npm run build && node iac test file.tf --experimental
  • Run ls .iac-data and see that there is no such directory in the filesystem anymore.

What are the relevant tickets?

CC-765

@github-actions
Copy link
Contributor

github-actions bot commented Mar 25, 2021

Fails
🚫

Commit " feat: delete local iac cache directory after tests" is not a valid commitizen message. See Contributing page with required commit message format.

Messages
📖

This PR will not trigger a new version. It doesn't include any commit message with feat or fix.

Generated by 🚫 dangerJS against bbb17be

@ipapast ipapast changed the title feat: Delete local iac cache directory after test finishes feat: Delete local .iac-data directory after scan is finished, using fs [CC-765] Mar 25, 2021
@ipapast ipapast force-pushed the refactor/clean-up-local-iac-cache-rmdirsync branch from ea5deed to bbb17be Compare March 25, 2021 21:13
@ipapast ipapast requested a review from aron March 26, 2021 09:43
@ipapast
Copy link
Contributor Author

ipapast commented Mar 26, 2021

Closing PR as rimraf was used instead #1762

@ipapast ipapast closed this Mar 26, 2021
@ipapast ipapast removed the request for review from aron March 26, 2021 14:06
@ipapast ipapast deleted the refactor/clean-up-local-iac-cache-rmdirsync branch April 5, 2021 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant