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.

Sign up for GitHub

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

Jump to bottom

[AA] refactor and add test #15882

Open

lightmark wants to merge 1 commit into main from lightmark/refactor_aa

+95 −44

Contributor

lightmark commented Feb 4, 2025

Description

refactor add authentication function.
add a test case returning an invalid signer.

How Has This Been Tested?

ut

Type of Change


          [AA] refactor and add test

6a2f448

lightmark requested review from gregnazario, runtian-zhou and igor-aptos

February 4, 2025 13:33

lightmark requested review from davidiw, movekevin and wrwg as code owners

February 4, 2025 13:33

trunk-io bot commented Feb 4, 2025 •

edited

Loading

⏱️ 5m total CI duration on this PR

Job	Cumulative Duration	Recent Runs
check-dynamic-deps	2m	🟩
rust-cargo-deny	2m	🟩
general-lints	26s	🟩
semgrep/ci	21s	🟩
file_change_determinator	14s	🟩
permission-check	4s	🟩
permission-check	2s	🟩

_{settings ⋅ feedback ⋅ docs ⋅ learn more about trunk.io}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

gregnazario Awaiting requested review from gregnazario

runtian-zhou Awaiting requested review from runtian-zhou

igor-aptos Awaiting requested review from igor-aptos

davidiw Awaiting requested review from davidiw davidiw is a code owner

movekevin Awaiting requested review from movekevin movekevin is a code owner

wrwg Awaiting requested review from wrwg wrwg is a code owner

At least 2 approving reviews are required to merge this pull request.

Labels

None yet