Skip to content

Commit

Permalink
Merge pull request #45 from mblottiere/fix/missing-problem-matcher
Browse files Browse the repository at this point in the history
fix: missing problem-matcher
  • Loading branch information
lorenzo committed Mar 22, 2022
2 parents 3cfc69d + 390bcfc commit 40f98da
Showing 1 changed file with 4 additions and 8 deletions.
12 changes: 4 additions & 8 deletions hadolint.sh
@@ -1,23 +1,19 @@
#!/bin/bash

# The problem-matcher definition must be present in the repository
# checkout (outside the Docker container running hadolint). We create
# a temporary folder and copy problem-matcher.json to it and make it
# readable.
TMP_FOLDER=$(mktemp -d -p .)
cp /problem-matcher.json "${TMP_FOLDER}"
chmod -R a+rX "${TMP_FOLDER}"
# checkout (outside the Docker container running hadolint). We copy
# problem-matcher.json to the home folder.
cp /problem-matcher.json "$HOME/"

# After the run has finished we remove the problem-matcher.json from
# the repository so we don't leave the checkout dirty. We also remove
# the matcher so it won't take effect in later steps.
cleanup() {
echo "::remove-matcher owner=brpaz/hadolint-action::"
rm -rf "${TMP_FOLDER}"
}
trap cleanup EXIT

echo "::add-matcher::${TMP_FOLDER}/problem-matcher.json"
echo "::add-matcher::$HOME/problem-matcher.json"

if [ -n "$HADOLINT_CONFIG" ]; then
HADOLINT_CONFIG="-c ${HADOLINT_CONFIG}"
Expand Down

0 comments on commit 40f98da

Please sign in to comment.