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.

Sign up for GitHub

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(external PR, twt--): Remove quotes $artifacts_to_transpile in the for loop list of compile_then_transpile.sh #8932

Merged
merged 3 commits into from
Oct 3, 2024
Merged

fix(external PR, twt--): Remove quotes $artifacts_to_transpile in the for loop list of compile_then_transpile.sh #8932

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
aa8e77c
Remove quotes on list of files to transpile in the transpiler script
twt-- Sep 27, 2024
675605d
Merge branch 'master' into fix-transpiling-multiple-artifacts
AztecBot Sep 27, 2024
50d20a7
Merge branch 'master' into chore/typo-redo-8845
ludamad Oct 2, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion avm-transpiler/scripts/compile_then_transpile.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,8 @@ artifacts_to_transpile=$($NARGO compile --show-artifact-paths $@ | tee /dev/tty
# If the script is run via docker, however, the user will see this output on stdout and will be able to redirect.

# Transpile each artifact
for artifact in "$artifacts_to_transpile"; do
# `$artifacts_to_transpile` needs to be unquoted here, otherwise it will break if there are multiple artifacts
for artifact in $artifacts_to_transpile; do
# transpiler input and output files are the same (modify in-place)
$TRANSPILER "$artifact" "$artifact"
done
Loading