Skip to content
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

Skip signing if the required secrets are not available #908

Merged
merged 5 commits into from
Jan 10, 2025
Merged

Conversation

Stefterv
Copy link
Collaborator

@Stefterv Stefterv commented Jan 10, 2025

This should fix the secrets not being available

closes #906

@Stefterv Stefterv requested a review from SableRaf January 10, 2025 16:45
@SableRaf
Copy link
Collaborator

Looks good to me!

Should properly set PROCESSING_APP_SIGNING to be handled by build.xml here:

<target name="macos-dist-sign" if="env.PROCESSING_APP_SIGNING">

@Stefterv
Copy link
Collaborator Author

Man, I was so confused by the error, but it's now failing the other way. Any checks that I trigger on a pull request do have the secret but are now skipped haha

@Stefterv
Copy link
Collaborator Author

Stefterv commented Jan 10, 2025

Turns out you can't if check secrets in GitHub Actions, the workarounds are too obnoxious so no signing on pull requests

@Stefterv
Copy link
Collaborator Author

closes #906

@SableRaf SableRaf merged commit 5f32f67 into main Jan 10, 2025
13 checks passed
@Stefterv Stefterv deleted the skip-signing branch January 10, 2025 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PR auto tests fail at "Install Cretificates for Code Signing" stage
2 participants