Skip to content

Improve shelling out in bin/changelog

Jacob Vosmaer requested to merge changelog-code-style into master

What does this MR do?

  • check exit status when capturing output of a shell command
  • don't use backticks (%x{}), in accordance with security guide lines that apply to the rest of this repo

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Edited by Jacob Vosmaer

Merge request reports

Loading