containers / qm

QM is a containerized environment for running Functional Safety qm (Quality Management) software
https://github.com/containers/qm
GNU General Public License v2.0
20 stars 20 forks source link

Fix again pre-commit dected issues and turn test on for all files #482

Closed sandrobonazzola closed 3 weeks ago

sandrobonazzola commented 3 weeks ago

Fix missing double quoting in tools/qm-storage-settings

Fixes missing double quotes introduced in #479. Not detected by pre-commit action as it's not configured to run on all files.

In tools/qm-storage-settings fix for:
     for STORAGE_CONF_PATH in ${STORAGE_CONF_PATHS[@]}; do
                                 ^----------------------^ SC2068 (error): Double quote array expansions to avoid re-splitting elements.

In tools/qm-storage-settings line 52:
        if [ -f ${STORAGE_CONF_PATH} ]; then
                ^------------------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Then, enabling pre-commit on all files preventing this to happen again.