ukaea / PROCESS

PROCESS is a systems code at UKAEA that calculates in a self-consistent manner the parameters of a fusion power plant with a specified performance, ensuring that its operating limits are not violated, and with the option to optimise to a given function of these parameters.
https://ukaea.github.io/PROCESS/
MIT License
26 stars 11 forks source link

[BUG] CI tracking step will fail if commit message contains `'` #3218

Open je-cook opened 3 weeks ago

je-cook commented 3 weeks ago

Describe the bug

If a commit has an apostrophe in the commit message the tracking script will have the remainder of the commit message passed to it as an extra argument. This is because the script is run from bash and bash needs strings escaped see link for a full list: https://unix.stackexchange.com/questions/270977/what-characters-are-required-to-be-escaped-in-command-line-arguments

Steps to reproduce

Add an apostrophe to you're commit message and the tracking step of the CI will fail

Expected behaviour

tracking step not fail on an extra apostrophe

Evidence

See https://github.com/ukaea/PROCESS/actions/runs/9386752431/job/25848590287

Environment

Additional context

This line in the CI yaml and the rest of that step will fail unless the commit message is escaped : https://github.com/ukaea/PROCESS/blob/6d0d9a51c12b449c1e351f74da50b779aa98e0eb/.github/workflows/process.yml#L248

possible solutions: https://stackoverflow.com/questions/2854655/command-to-escape-a-string-in-bash