-
-
Notifications
You must be signed in to change notification settings - Fork 87
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GitHub Actions CI triggered by PRs by others fails #164
Comments
Author's key is not passed to PR builds (shimataro#164)
#143 must be fixed before using https://github.com/linuxserver/docker-openssh-server because its host key changes every after containers created |
* Ignore Node files that should not be tracked The list can be fetched by `gibo dump node`. * Untrack node_modules/@actions * Cache node_modules * Don't add node_modules ins Bash scripts * Use ncc to pack dependencies * Change final product path Remove & ignore previous one (lib/main.js{,.map}) * Disable PR check using author's key Author's key is not passed to PR builds (#164) * update settings * update CHANGELOG * update build.yml Co-authored-by: Tatsunori Uchino <[email protected]>
I think it cannot be tested on Windows/macOS if I use Docker service container... 🤔 |
Windows can be tested because Docker is installed. |
@tats-u
I want to guarantee this action to work on all available virtual machines definitely. It is tested on various versions of Ubuntu for that. I will consider using Docker when macOS supports it. Until then, I handle PRs in the same way as #156 / #163. |
@shimataro |
Encrypted secrets in GitHub Actions are not passed to builds triggered by forked repositories (PRs from others).
https://docs.github.com/ja/free-pro-team@latest/actions/reference/encrypted-secrets#%E6%9A%97%E5%8F%B7%E5%8C%96%E3%81%95%E3%82%8C%E3%81%9F%E3%82%B7%E3%83%BC%E3%82%AF%E3%83%AC%E3%83%83%E3%83%88%E3%81%AE%E3%83%AF%E3%83%BC%E3%82%AF%E3%83%95%E3%83%AD%E3%83%BC%E5%86%85%E3%81%A7%E3%81%AE%E5%88%A9%E7%94%A8 (Japanese)
This makes
Connect to github.com
andConnect to github.com with name and config
for PRs fail.Take
/.github/workflows/verify-on-ubuntu-2004.yml
as an example.ssh-key-action/.github/workflows/verify-on-ubuntu-2004.yml
Lines 11 to 20 in 66374ea
In this configuration,
key
andknown_hosts
are ignored in PRs (except for yours). Options passed to builds for my PR are:name: id_rsa
is the default option, so you can see those 2 options are missing.It may be fixed by changing the SSH target to one created by Docker (you do not have to apply it to builds for
develop
branch).Image candidate: https://github.com/linuxserver/docker-openssh-server
The text was updated successfully, but these errors were encountered: