diff --git a/.github/workflows/check_samples.yml b/.github/workflows/check_samples.yml index af4c79b..3ff7c56 100644 --- a/.github/workflows/check_samples.yml +++ b/.github/workflows/check_samples.yml @@ -2,54 +2,54 @@ name: Check Samples on: push: - branches: [ master ] + branches: [master] pull_request: - branches: [ master ] + branches: [master] jobs: check-allowed-file-names: runs-on: ubuntu-latest steps: - - uses: actions/checkout@v2 + - uses: actions/checkout@v3 - - name: Check Allowed File Names - run: | - NOT_SAMPLES=$(find . -not -path '*/\.*' -type f ! \( -name '*.conf.sample' -o -name 'README.md' -o -name 'LICENSE' \)) - NOT_SAMPLES_COUNT=$(echo "${NOT_SAMPLES}" | wc -w) - if (( NOT_SAMPLES_COUNT > 0 )); then - echo "The following files have extensions that are not allowed:" - echo "${NOT_SAMPLES}" - exit 1 - fi + - name: Check Allowed File Names + run: | + NOT_SAMPLES=$(find . -not -path '*/\.*' -type f ! \( -name '*.conf.sample' -o -name 'README.md' -o -name 'LICENSE' \)) + NOT_SAMPLES_COUNT=$(echo "${NOT_SAMPLES}" | wc -w) + if (( NOT_SAMPLES_COUNT > 0 )); then + echo "The following files have extensions that are not allowed:" + echo "${NOT_SAMPLES}" + exit 1 + fi - - name: Check Executable Bit - run: | - EXECUTABLE_BIT=$(find . -not -path '*/\.*' -type f -executable) - EXECUTABLE_BIT_COUNT=$(echo "${EXECUTABLE_BIT}" | wc -w) - if (( EXECUTABLE_BIT_COUNT > 0 )); then - echo "The following files have executable permissions (not allowed):" - echo "${EXECUTABLE_BIT}" - exit 1 - fi + - name: Check Executable Bit + run: | + EXECUTABLE_BIT=$(find . -not -path '*/\.*' -type f -executable) + EXECUTABLE_BIT_COUNT=$(echo "${EXECUTABLE_BIT}" | wc -w) + if (( EXECUTABLE_BIT_COUNT > 0 )); then + echo "The following files have executable permissions (not allowed):" + echo "${EXECUTABLE_BIT}" + exit 1 + fi - - name: Check Line Endings - run: | - CRLF_ENDINGS=$(find . -not -path '*/\.*' -type f -exec file "{}" ";" | grep CRLF || true) - CRLF_ENDINGS_COUNT=$(echo "${CRLF_ENDINGS}" | wc -w) - if (( CRLF_ENDINGS_COUNT > 0 )); then - echo "The following files have CRLF line endings (not allowed):" - echo "${CRLF_ENDINGS}" - exit 1 - fi + - name: Check Line Endings + run: | + CRLF_ENDINGS=$(find . -not -path '*/\.*' -type f -exec file "{}" ";" | grep CRLF || true) + CRLF_ENDINGS_COUNT=$(echo "${CRLF_ENDINGS}" | wc -w) + if (( CRLF_ENDINGS_COUNT > 0 )); then + echo "The following files have CRLF line endings (not allowed):" + echo "${CRLF_ENDINGS}" + exit 1 + fi - - name: Check Version Date Line Exists - run: | - # Date regex based on https://www.html5pattern.com/Dates - VERSION_LINE_MISSING=$(find . -not -path '*/\.*' -type f -name '*.conf.sample' -exec grep -H -c -P '^## Version (?:19|20|21)[0-9]{2}/(?:(?:0[1-9]|1[0-2])/(?:0[1-9]|1[0-9]|2[0-9])|(?:(?!02)(?:0[1-9]|1[0-2])/(?:30))|(?:(?:0[13578]|1[02])/31))$' {} \; | grep 0$ | cut -d':' -f1) - VERSION_LINE_MISSING_COUNT=$(echo "${VERSION_LINE_MISSING}" | wc -w) - if (( VERSION_LINE_MISSING_COUNT > 0 )); then - echo "The following files are missing the version date line or it is not formatted correctly (YYYY/MM/DD):" - echo "${VERSION_LINE_MISSING}" - exit 1 - fi + - name: Check Version Date Line Exists + run: | + # Date regex based on https://www.html5pattern.com/Dates + VERSION_LINE_MISSING=$(find . -not -path '*/\.*' -type f -name '*.conf.sample' -exec grep -H -c -P '^## Version (?:19|20|21)[0-9]{2}/(?:(?:0[1-9]|1[0-2])/(?:0[1-9]|1[0-9]|2[0-9])|(?:(?!02)(?:0[1-9]|1[0-2])/(?:30))|(?:(?:0[13578]|1[02])/31))$' {} \; | grep 0$ | cut -d':' -f1) + VERSION_LINE_MISSING_COUNT=$(echo "${VERSION_LINE_MISSING}" | wc -w) + if (( VERSION_LINE_MISSING_COUNT > 0 )); then + echo "The following files are missing the version date line or it is not formatted correctly (YYYY/MM/DD):" + echo "${VERSION_LINE_MISSING}" + exit 1 + fi