chore: fix proto script file handling and safety #977
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.
()
aroundgo install …
to avoid spawning a useless subshell..proto
files with spaces or special characters:for file in $(find …)
→find … -print0 | while IFS= read -r -d '' file
.grep "option go_package"
togrep -q "option go_package" "$file"
for quiet search and proper quoting.cd proto || exit 1
to stop the script if the directory doesn’t exist.x/
directory before copying files to prevent errors:mkdir -p x
.rm -rf
to only remove the target subdirectory (github.com/neutron-org
) instead of the wholegithub.com
.changes make the script safer, more robust, and prevent accidental deletion or errors when handling files with tricky names.