fix: removeSync in case of directory in copy-to-clients #1452
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue #, if available:
Internal issue JS-2016
Description of changes:
removeSync in case of directory in copy-to-clients
As detailed in internal issue JS-2016, the current copy-to-clients script doesn't remove commands which could have been removed from models. This PR ensures that the directories are deleted before individual files are copied over. The command
ensureDirSync
doesn't need to be run, ascopySync
would create the directory.Testing:
Verified that the files
GetPredictionCommand.ts
andPutModelCommand.ts
were removed while generating client for frauddetector using updated models.By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.