Skip to content

Commit 70ccedc

Browse files
takecyclaude
andcommitted
fix(syncer): drop trailing newline from Success message to avoid blank line
The Success message used "Success: %s\n" while Timeout and Failed used the trailing-newline-free form. Combined with msgTpl's own "\n", the explicit \n in the format string produced an empty line right after every successful repo. The output already includes a visible separator from the Print template, so the extra blank line was pure noise. Drop the \n from the format string. Output now matches the Timeout and Failed shapes: a single line ending in the template's trailing newline. Reported by GitHub Copilot on PR #26. Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 7d420ad commit 70ccedc

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

syncer/syncer.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -194,7 +194,7 @@ func (s *Sync) execute(parent context.Context, repos []string, perRepoTimeout ti
194194
switch {
195195
case err == nil:
196196
stats.addSuccess(r)
197-
s.Writer.PrintMsg(fmt.Sprintf("Success: %s\n", r))
197+
s.Writer.PrintMsg(fmt.Sprintf("Success: %s", r))
198198
case errors.Is(ctx.Err(), context.DeadlineExceeded):
199199
stats.addTimedOut(r)
200200
s.Writer.PrintMsgErr(fmt.Sprintf("Timeout: %s", r))

0 commit comments

Comments
 (0)