luci-app-upnp: Add Expires
to port map listing
#4811
Workflow file for this run
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
name: Test Formalities | |
on: | |
pull_request: | |
jobs: | |
build: | |
name: Test Formalities | |
runs-on: ubuntu-latest | |
strategy: | |
fail-fast: false | |
steps: | |
- uses: actions/checkout@v4 | |
with: | |
ref: ${{ github.event.pull_request.head.sha }} | |
fetch-depth: 0 | |
- name: Determine branch name | |
run: | | |
BRANCH="${GITHUB_BASE_REF#refs/heads/}" | |
echo "Building for $BRANCH" | |
echo "BRANCH=$BRANCH" >> $GITHUB_ENV | |
- name: Test for merge commits, subject, S.O.B., and email | |
run: | | |
source .github/workflows/ci_helpers.sh | |
RET=0 | |
for commit in $(git rev-list HEAD ^origin/$BRANCH); do | |
info "=== Checking commit '$commit'" | |
if git show --format='%P' -s $commit | grep -qF ' '; then | |
err "Pull request should not include merge commits" | |
RET=1 | |
fi | |
authorname="$(git show -s --format=%aN $commit)" | |
if echo $authorname | grep -q '\S\+\s\+\S\+'; then | |
success "Author name ($authorname) seems ok" | |
elif echo $authorname | grep -q '\S\+'; then | |
success "Author name ($authorname) seems to be nick or alias" | |
else | |
err "Author name ($authorname) must be one of: real name 'firstname lastname' OR nickname/alias/handle " | |
RET=1 | |
fi | |
committername="$(git show -s --format=%cN $commit)" | |
# Pattern \S\+\s\+\S\+ matches >= 2 names i.e. 3 and more e.g. "John Von Doe" also match | |
if echo $committername | grep -q '\S\+\s\+\S\+'; then | |
success "Committer name ($committername) seems ok" | |
elif echo $committername | grep -q '\S\+'; then | |
# Pattern \S\+ matches single names, typical of nicks or handles | |
success "Committer name ($committername) seems to be nick or alias" | |
else | |
err "Committer name ($committername) must be one of: real name 'firstname lastname' OR nickname/alias/handle " | |
RET=1 | |
fi | |
subject="$(git show -s --format=%s $commit)" | |
if echo "$subject" | grep -q -e '^[0-9A-Za-z,+/_-]\+: ' -e '^Revert '; then | |
success "Commit subject line seems ok ($subject)" | |
elif echo "$subject" | grep -iq '^Translated using Weblate.*'; then | |
success "Weblate commit subject line exception OK: $subject" | |
elif echo "$subject" | grep -iq '^Added translation using Weblate.*'; then | |
success "Weblate commit subject line exception OK: $subject" | |
else | |
err "Commit subject line MUST start with '<package name>: ' ($subject)" | |
RET=1 | |
fi | |
body="$(git show -s --format=%b $commit)" | |
authoremail="$(git show -s --format='<%aE>' $commit)" | |
sob="$(git show -s --format='Signed-off-by: %aN <%aE>' $commit)" | |
if echo "$body" | grep -qF "$sob"; then | |
success "Signed-off-by matches author" | |
elif echo "$authoremail" | grep -iqF "<hosted@weblate.org>"; then | |
success "Signed-off-by exception: authored by Weblate" | |
else | |
err "Signed-off-by is missing or doesn't match author (should be '$sob')" | |
RET=1 | |
fi | |
if echo "$authoremail" | grep -iqF "users.noreply"; then | |
err "Real email address policy: please configure GitHub email ($authoremail) to a real one" | |
RET=1 | |
fi | |
done | |
exit $RET |