Avoid adding a redundant newline when printing weblistPageClosing

[zpavlinovic]

weblistPageClosing string ends in a new line, so printing it via fmt.Println will add an extra new line.

[aalexand]

Could you elaborate why is this change needed? What's wrong with Println?

[aalexand]

Could you elaborate why is this change needed? What's wrong with Println?

If this is to avoid a redundant newline, then I think it's easier to update the value of the weblistPageClosing var to remove it there and leave the code as is.

And also the PR description needs to be updated I think. I don't think this change has anything to do with the linked 'go vet' changes. Those changes are about format strings for Printf.

[zpavlinovic]

Could you elaborate why is this change needed? What's wrong with Println?

If this is to avoid a redundant newline, then I think it's easier to update the value of the weblistPageClosing var to remove it there and leave the code as is.

Sounds good.

And also the PR description needs to be updated I think. I don't think this change has anything to do with the linked 'go vet' changes. Those changes are about format strings for Printf.

Yes, it is about avoiding a redundant new line. The reason I mentioned vet is because a previous limitation has been recently addressed. I will change the description.

[aalexand]

The description still mentions "go vet".

[zpavlinovic]

The description still mentions "go vet".

Hopefully, I have corrected it at all the relevant places now.

[codecov-commenter]

Codecov Report

Merging #665 (0b0dd28) into master (f410f49) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #665   +/-   ##
=======================================
  Coverage   68.19%   68.19%           
=======================================
  Files          41       41           
  Lines        7391     7391           
=======================================
  Hits         5040     5040           
  Misses       1910     1910           
  Partials      441      441           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f410f49...0b0dd28. Read the comment docs.