Skip to content

Commit ad018ca

Browse files
committed
deploy: do catch non-HTTPS links to git-scm.com
After merging #1953, as per https://github.com/git/git-scm.com/actions/runs/13539600621/job/37837557415#step:3:135167 the `deploy` workflow failed. This issue should have been found in a PR build, but was not. Let's make sure that similar issues will be found early in the future, before merging the respective PR. Signed-off-by: Johannes Schindelin <[email protected]>
1 parent c6a19e3 commit ad018ca

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

.github/actions/deploy-to-github-pages/action.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,10 @@ runs:
141141
# A simple `grep` should work without any false positives,
142142
# unless git-scm.com mentions the base URL of one of its forks,
143143
# which is unlikely.
144-
run: '! grep -FInr "http:${base_url#https:}" public'
144+
#
145+
# To catch bugs early, let's always look for non-HTTPS links
146+
# to git-scm.com.
147+
run: '! grep -FInre "http://git-scm.com" -e "http:${base_url#https:}" public'
145148

146149
- name: check for broken links
147150
id: lychee

0 commit comments

Comments
 (0)