Skip to content

Add ssh-key input for SSH-based deployments#134

Closed
npwolf wants to merge 1 commit intorossjrw:mainfrom
datagrail:add-ssh-key
Closed

Add ssh-key input for SSH-based deployments#134
npwolf wants to merge 1 commit intorossjrw:mainfrom
datagrail:add-ssh-key

Conversation

@npwolf
Copy link
Copy Markdown

@npwolf npwolf commented Mar 4, 2026

Summary

Passes through an ssh-key input to JamesIves/github-pages-deploy-action for both deploy and remove steps. This allows users to authenticate via a deploy key instead of a token, which is useful for cross-repository deployments or environments where SSH is preferred.

The input is optional and is a no-op when not provided, preserving existing behavior.

Changes

  • Added ssh-key input to action.yml
  • Passed ssh-key to both the deploy and remove invocations of github-pages-deploy-action

Context

We've been maintaining this as a fork addition at datagrail/pr-preview-action and would like to contribute it upstream so we can stop carrying the patch.

Pass through an `ssh-key` input to JamesIves/github-pages-deploy-action
for both deploy and remove steps. This allows users to authenticate via
a deploy key instead of a token, which is useful for cross-repository
deployments or environments where SSH is preferred.

The input is optional and is a no-op when not provided, preserving
existing behavior.
@npwolf npwolf closed this Mar 4, 2026
@npwolf npwolf deleted the add-ssh-key branch March 4, 2026 20:06
@rossjrw
Copy link
Copy Markdown
Owner

rossjrw commented Mar 5, 2026

Seems like a fair feature request, was the PR closure / branch deletion intentional?

@npwolf
Copy link
Copy Markdown
Author

npwolf commented Mar 5, 2026

Seems like a fair feature request, was the PR closure / branch deletion intentional?

Yeah, this worked previously, but after syncing our fork it broke and I decided to just move to a github app.

@rossjrw
Copy link
Copy Markdown
Owner

rossjrw commented Mar 6, 2026

Interesting - I can't imagine why that would have been affected at all. Would be happy to help you figure it out if you're able to share more info / workflow logs, but if you've already figured out a different solution then no worries.

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