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: update bytebase-sql-review.yml for CICD #8984
Conversation
Tthe `echo` command is wrapped in double quotes to preserve the JSON structure stored in the `content` variable. in some case, it'll return ``` sh: 0: unknown operand ``` and cause script run incorrectly
Tthe `echo` command is wrapped in double quotes to preserve the JSON structure stored in the `content` variable. in some case, it'll return ``` sh: 0: unknown operand ``` and cause script run incorrectly
Update bytebase-sql-review.yml
We require contributors to sign our Contributor License Agreement, and we don't have yours on file. In order for us to review and merge your code, please please sign CLA and add your name to contributors list. |
LGTM, please sign the cla. |
@cla-bot check. |
The GitHub CLA Bot is rechecking to see that you have signed the CLA |
already signed CLA, can you help me recheck CLA? |
Thanks! |
The change has been cherry-picked to 2.10.0. |
Why:
Sometimes, the pipeline return sql review passed but when deploy on Bytebase, the sql checking (UI) return some warning/error check.
We re-review log in cicd and noticed that script return
that cause next steps in script will always failed because there is no data.
Fix:
The
echo
command is wrapped indouble quotes
to preserve the JSON structure stored in thecontent
variable.in some case, it'll return
and cause script run incorrectly