Skip to content

Remove bogus '|' symbol

Jacob Vosmaer requested to merge remove-pipe into master

What does this MR do?

This removes a weird | argument from a Git command argument list. To our best understanding this argument gets ignored by Git. Removing it is good because it looks suspiciously like a shell pipe, so it stands out as a potential security vulnerability. It isn't one because it doesn't get parsed by a shell. Removing it makes this line less confusing.

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?

Merge request reports