From 8d374c7cb51e320dc547b554d6712409cccd6b09 Mon Sep 17 00:00:00 2001 From: Thomas Heritage Date: Thu, 12 Mar 2026 12:38:07 +0000 Subject: [PATCH 1/3] action.yml: Configure git to use HTTPS for GitHub repos build.mjs runs a number of git commands including cloning repos (and their submodules). Any operation using SSH fails because SSH is not configured on the runner. For example, submodules might be configured using SSH (with "git@github.com" in ".gitmodules"). This commit configures git so that all uses of SSH for GitHub repos are automatically replaced with HTTPS. This is comparable to the behaviour of https://github.com/actions/checkout which notes "When the `ssh-key` input is not provided, SSH URLs beginning with `git@github.com:` are converted to HTTPS" This is added to action.yml rather than to build.mjs because it aims to configure the runner environment -- if build.mjs is run in other contexts (e.g. local machine) then a different configuration might be needed. --- workflows/action.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/workflows/action.yml b/workflows/action.yml index 175a993..ffae253 100644 --- a/workflows/action.yml +++ b/workflows/action.yml @@ -35,6 +35,10 @@ runs: shell: bash run: npm install ${GITHUB_ACTION_PATH}/.. + - name: Configure git to use HTTPS for GitHub repos + shell: bash + run: git config --global --replace-all url."https://github.com/".insteadOf "git@github.com:" + - name: Build and deploy spec id: build shell: bash From 1c99c50f7f1576e1ea11ee572b0542c291afa1a5 Mon Sep 17 00:00:00 2001 From: Steve LLamb <38917682+SteveLLamb@users.noreply.github.com> Date: Tue, 17 Mar 2026 14:27:37 -0700 Subject: [PATCH 2/3] remove warning gate, closes #391 --- scripts/build.mjs | 16 ++++------------ 1 file changed, 4 insertions(+), 12 deletions(-) diff --git a/scripts/build.mjs b/scripts/build.mjs index 1b6191c..e3a76e2 100644 --- a/scripts/build.mjs +++ b/scripts/build.mjs @@ -192,12 +192,8 @@ async function build(buildPaths, baseRef, lastEdRef, docMetadata) { console.log(`Generating a redline against base: ${baseRef}.`); - try { - await generateRedline(buildPaths, baseRef, buildPaths.baseRedLineRefPath, buildPaths.baseRedlinePath); - generatedFiles.baseRedline = buildPaths.baseRedlineName; - } catch (e) { - console.warn(`Could not generate a redline: ${e}.`); - } + await generateRedline(buildPaths, baseRef, buildPaths.baseRedLineRefPath, buildPaths.baseRedlinePath); + generatedFiles.baseRedline = buildPaths.baseRedlineName; } @@ -207,12 +203,8 @@ async function build(buildPaths, baseRef, lastEdRef, docMetadata) { console.log(`Generating a redline against the latest edition tag: ${lastEdRef}.`); - try { - await generateRedline(buildPaths, lastEdRef, buildPaths.pubRedLineRefPath, buildPaths.pubRedlinePath); - generatedFiles.pubRedline = buildPaths.pubRedlineName; - } catch (e) { - console.warn(`Could not generate a redline: ${e}.`); - } + await generateRedline(buildPaths, lastEdRef, buildPaths.pubRedLineRefPath, buildPaths.pubRedlinePath); + generatedFiles.pubRedline = buildPaths.pubRedlineName; } From 1c41f30867bf11144a4cbed3423ff788e995a191 Mon Sep 17 00:00:00 2001 From: Steve LLamb <38917682+SteveLLamb@users.noreply.github.com> Date: Tue, 17 Mar 2026 15:02:33 -0700 Subject: [PATCH 3/3] undo commit to push to new PR --- scripts/build.mjs | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/scripts/build.mjs b/scripts/build.mjs index e3a76e2..1b6191c 100644 --- a/scripts/build.mjs +++ b/scripts/build.mjs @@ -192,8 +192,12 @@ async function build(buildPaths, baseRef, lastEdRef, docMetadata) { console.log(`Generating a redline against base: ${baseRef}.`); - await generateRedline(buildPaths, baseRef, buildPaths.baseRedLineRefPath, buildPaths.baseRedlinePath); - generatedFiles.baseRedline = buildPaths.baseRedlineName; + try { + await generateRedline(buildPaths, baseRef, buildPaths.baseRedLineRefPath, buildPaths.baseRedlinePath); + generatedFiles.baseRedline = buildPaths.baseRedlineName; + } catch (e) { + console.warn(`Could not generate a redline: ${e}.`); + } } @@ -203,8 +207,12 @@ async function build(buildPaths, baseRef, lastEdRef, docMetadata) { console.log(`Generating a redline against the latest edition tag: ${lastEdRef}.`); - await generateRedline(buildPaths, lastEdRef, buildPaths.pubRedLineRefPath, buildPaths.pubRedlinePath); - generatedFiles.pubRedline = buildPaths.pubRedlineName; + try { + await generateRedline(buildPaths, lastEdRef, buildPaths.pubRedLineRefPath, buildPaths.pubRedlinePath); + generatedFiles.pubRedline = buildPaths.pubRedlineName; + } catch (e) { + console.warn(`Could not generate a redline: ${e}.`); + } }