tangled.org / core
this repo has no description
fork atom

knotserver: always show pipeline errors

IMO, we shouldn't make errors opt-in to actually show upon push, at least
not if/until we have a UI to show them to the user.

Signed-off-by: Winter <winter@winter.cafe>

authored by Winter and committed by Tangled d3e7d517 c9d11d54

options
unified split
Changed files
+4 -4
knotserver
internal.go
+4 -4
knotserver/internal.go
··· 242 return err 243 } 244 245 if pushOptions.verboseCi { 246 if compiler.Diagnostics.IsEmpty() { 247 *clientMsgs = append(*clientMsgs, "success: pipeline compiled with no diagnostics") 248 - } 249 - 250 - for _, e := range compiler.Diagnostics.Errors { 251 - *clientMsgs = append(*clientMsgs, e.String()) 252 } 253 254 for _, w := range compiler.Diagnostics.Warnings {
··· 242 return err 243 } 244 245 + for _, e := range compiler.Diagnostics.Errors { 246 + *clientMsgs = append(*clientMsgs, e.String()) 247 + } 248 + 249 if pushOptions.verboseCi { 250 if compiler.Diagnostics.IsEmpty() { 251 *clientMsgs = append(*clientMsgs, "success: pipeline compiled with no diagnostics") 252 } 253 254 for _, w := range compiler.Diagnostics.Warnings {