-
Notifications
You must be signed in to change notification settings - Fork 4
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
Signature check #4
base: development
Are you sure you want to change the base?
Conversation
Coverage Report (0%)
|
Can you make sure that CI passes? Alternatively make a new task for it. |
Before merging, @whilefoo could you pass the the public key, or could you set it yourself the in Action Env? Thanks. |
I'm not sure if I have access to the private key and I don't know which github app the new bot uses right now |
Can try using @gentlementlegen v2 testing bot. But yeah should be an env var |
Can be any dummy value as long as it works with the new bot and new plugins to get valid signatures, you can DM it to me and I'll update it if you want. |
Currently the public key for the ubiquibot github app is retrieved from the env but later this should be a part of the SDK (or at least plugin starter template) so plugin developers don't have to worry about that