feature/21176 #1

Merged
roberto.cirillo merged 10 commits from feature/21176 into master 2021-11-18 16:15:02 +01:00
1 changed files with 7 additions and 3 deletions
Showing only changes of commit 15a482bd26 - Show all commits

10
Jenkinsfile vendored
View File

@ -183,12 +183,16 @@ def printReport(report) {
text += "\n"
}
println text
sh("""
printf "%s" $text > $WALKER_NOTES
""")
appendNotes(text)
}
manuele.simi marked this conversation as resolved
Review

You don't need the shebang here.

You don't need the shebang here.
Review

Are you referring to the bash shell?

I've used the bash shell otherwise the echo -e option doesn't work.

Is there an easier way to do that?

Are you referring to the bash shell? I've used the bash shell otherwise the `echo -e` option doesn't work. Is there an easier way to do that?
Review

echo (without -e) works for sure without the shebang.

But if it does work for you, leave like that. It's a detail.

`echo` (without -e) works for sure without the shebang. But if it does work for you, leave like that. It's a detail.
Review

echo (without -e) works for sure without the shebang.

But if it does work for you, leave like that. It's a detail.

Yes it works but without the "-e" option the backslashes are not escapes. The result is an unreadable file.

> `echo` (without -e) works for sure without the shebang. > > But if it does work for you, leave like that. It's a detail. > Yes it works but without the "-e" option the backslashes are not escapes. The result is an unreadable file.
def appendNotes(report) {
sh("""
echo -e "${report}" > $WALKER_NOTES
""")
}
// print the final report
def saveReport(report) {
def text = ''