Skip to content

Commit aa66e94

Browse files
committed
Handle connector docs rename validation
1 parent 53530da commit aa66e94

2 files changed

Lines changed: 8 additions & 2 deletions

File tree

.github/workflows/connector-docs-verify.yaml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,19 +30,25 @@ jobs:
3030
3131
API_URL="${GITHUB_API_URL:-https://api.github.com}"
3232
PAGE=1
33+
PAGE_LIMIT=30
3334
SHOULD_VALIDATE=false
3435
while true; do
3536
FILES=$(curl -fsSL \
3637
-H "Authorization: Bearer $GH_TOKEN" \
3738
-H "Accept: application/vnd.github+json" \
3839
"${API_URL}/repos/${REPOSITORY}/pulls/${PR_NUMBER}/files?per_page=100&page=${PAGE}")
3940
40-
if echo "$FILES" | jq -e '.[] | select(.filename == "docs/connector.mdx")' >/dev/null; then
41+
if echo "$FILES" | jq -e '.[] | select(.filename == "docs/connector.mdx" or .previous_filename == "docs/connector.mdx")' >/dev/null; then
4142
SHOULD_VALIDATE=true
4243
break
4344
fi
4445
4546
FILE_COUNT=$(echo "$FILES" | jq 'length')
47+
if [ "$PAGE" -ge "$PAGE_LIMIT" ] && [ "$FILE_COUNT" -eq 100 ]; then
48+
echo "::warning::PR file list reached the GitHub API cap; validating docs/connector.mdx because changed files cannot be proven complete."
49+
SHOULD_VALIDATE=true
50+
break
51+
fi
4652
if [ "$FILE_COUNT" -lt 100 ]; then
4753
break
4854
fi

tools/mdx-lint/mdx-lint.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ function containsDangerousUrl(input) {
6565
return (
6666
normalized.includes("javascript:") ||
6767
normalized.includes("vbscript:") ||
68-
normalized.includes("data:text/html")
68+
normalized.includes("data:")
6969
);
7070
}
7171

0 commit comments

Comments
 (0)