-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
chore: fix semantic expression of BLS cryptography #3927
base: main
Are you sure you want to change the base?
Conversation
@wonderfan is attempting to deploy a commit to the unionbuild Team on Vercel. A member of the Team first needs to authorize it. |
Hi, thanks for the PR! Please remove the merge commit and rebase your branch on main instead. See our CONTRIBUTING.md for all PR requirements. |
@benluelo Thanks for your kind remind. The change set are rebased and force pushed. Please take a look. |
hey @wonderfan, you will also need to sign your commits |
How to sign? |
This is described in our CONTRIBUTING.md: https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits |
b1003e4
to
f4d98e7
Compare
@benluelo Thanks for your helpful instruction. The code snippets are signed. Please take a look again. |
@aeryz @benluelo The failed checks have this error:
|
CLOUDFLARE_API_TOKEN should be set in the environment variable. |
The expression of BLS signatures and verifications is wrong and make them correct.