Skip to content

fix: consolidate support URL validation into single script#6870

Open
Copilot wants to merge 1 commit intomasterfrom
copilot/create-validation-change-pr
Open

fix: consolidate support URL validation into single script#6870
Copilot wants to merge 1 commit intomasterfrom
copilot/create-validation-change-pr

Conversation

Copy link
Contributor

Copilot AI commented Feb 25, 2026

Summary

Add support URL validation that runs in both local pre-commit hooks (LeftHook) and CI (GitHub Actions), enforcing two allowed URLs:

  • https://support.influxdata.com (Support home)
  • https://support.influxdata.com/s/contactsupport (Contact Support)

All other support.influxdata.com variations (trailing slashes, login redirects, other paths) are rejected.

Changes

File Change
.ci/scripts/check-support-links.sh New script — accepts file args (LeftHook/CI) or scans all content/**/*.md
lefthook.yml Add check-support-links pre-commit command calling the script
.github/workflows/pr-link-check.yml Add "Validate support links" step (runs before Node.js setup, no build needed)
DOCS-CONTRIBUTING.md Add InfluxData Support links section documenting both allowed URLs

Context

Extracted from PR #6867. Replaces three earlier redundant checks (dead Vale rule, inline LeftHook grep, standalone script) with a single script callable from both LeftHook and CI.

Test plan

  • Run .ci/scripts/check-support-links.sh on a file with https://support.influxdata.com/ — should fail
  • Run on a file with https://support.influxdata.com/s/contactsupport — should pass
  • Run on a file with https://support.influxdata.com — should pass
  • Verify CI workflow runs the validation step on PR content changes

Replace three redundant checks (Vale rule, inline LeftHook grep,
standalone script) with one script callable from both LeftHook and CI.

- Update check-support-links.sh to accept file args and allow both
  https://support.influxdata.com and .../s/contactsupport
- Add separate LeftHook command replacing inline grep
- Add CI step in pr-link-check.yml before Node.js setup
- Delete dead SupportLink.yml Vale rule (TokenIgnores strips URLs)
- Update DOCS-CONTRIBUTING.md with both allowed URLs
@jstirnaman jstirnaman force-pushed the copilot/create-validation-change-pr branch from ff6e5ee to 5f47d3d Compare February 27, 2026 04:24
@jstirnaman jstirnaman changed the title [WIP] Create separate PR for validation change fix: consolidate support URL validation into single script Feb 27, 2026
@jstirnaman jstirnaman marked this pull request as ready for review February 27, 2026 04:26
@jstirnaman jstirnaman requested a review from a team as a code owner February 27, 2026 04:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants