This is an automated email from the ASF dual-hosted git repository. linkinstar pushed a commit to branch dev in repository https://gitbox.apache.org/repos/asf/answer.git
The following commit(s) were added to refs/heads/dev by this push: new 30bac18a fix: add signal exit on build failure 30bac18a is described below commit 30bac18a85555dc9acfd18bce1783fe3f1eb8d7a Author: Luffy <5...@qq52o.cn> AuthorDate: Fri Apr 25 14:38:04 2025 +0800 fix: add signal exit on build failure --- cmd/command.go | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/cmd/command.go b/cmd/command.go index 42a404fb..fe17713e 100644 --- a/cmd/command.go +++ b/cmd/command.go @@ -227,10 +227,11 @@ To run answer, use: Time: Time, }) if err != nil { - fmt.Printf("build failed %v", err) - } else { - fmt.Printf("build new answer successfully %s\n", buildOutput) + fmt.Printf("build failed %v\n", err) + os.Exit(1) } + + fmt.Printf("build new answer successfully %s\n", buildOutput) }, } @@ -292,7 +293,7 @@ To run answer, use: Long: `Merge i18n files from plugins to original i18n files. It will overwrite the original i18n files`, Run: func(_ *cobra.Command, _ []string) { if err := cli.ReplaceI18nFilesLocal(i18nTargetPath); err != nil { - fmt.Printf("replace i18n files failed %v", err) + fmt.Printf("replace i18n files failed %v\n", err) } else { fmt.Printf("replace i18n files successfully\n") } @@ -300,7 +301,7 @@ To run answer, use: fmt.Printf("try to merge i18n files from %q to %q\n", i18nSourcePath, i18nTargetPath) if err := cli.MergeI18nFilesLocal(i18nTargetPath, i18nSourcePath); err != nil { - fmt.Printf("merge i18n files failed %v", err) + fmt.Printf("merge i18n files failed %v\n", err) } else { fmt.Printf("merge i18n files successfully\n") }