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.

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

Remove LdProofs.VerifyProof #425

Merged
merged 4 commits into from
May 18, 2022
Merged

Remove LdProofs.VerifyProof #425

merged 4 commits into from
May 18, 2022

Conversation

fundthmcalculus
Copy link
Contributor

closes #424

@fundthmcalculus fundthmcalculus linked an issue May 18, 2022 that may be closed by this pull request
Remove LdProofs.verify_proof #424
Closed
@github-actions
Copy link
Contributor

github-actions bot commented May 18, 2022 •
edited
Loading

Golang Test Report

  3 files  ±0    3 suites  ±0   0s ⏱️ ±0s
15 tests ±0  15 ✔️ ±0  0 💤 ±0  0 ❌ ±0 
45 runs  ±0  45 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit b997ab5. ± Comparison against base commit 2abe3bc.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented May 18, 2022 •
edited
Loading

.NET Test Report

0 files  ±0  0 suites  ±0   0s ⏱️ ±0s
0 tests ±0  0 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit b997ab5. ± Comparison against base commit 2abe3bc.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented May 18, 2022 •
edited
Loading

Java Test Report

12 files  ±0  12 suites  ±0   1s ⏱️ ±0s
18 tests ±0  18 ✔️ ±0  0 💤 ±0  0 ❌ ±0 
42 runs  ±0  42 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit b997ab5. ± Comparison against base commit 2abe3bc.

This pull request removes 6 and adds 6 tests. Note that renamed tests count towards both. 
OkapiNativeTest ‑ [1] OkapiNativeTest$DataArgument@41e68d87
OkapiNativeTest ‑ [1] OkapiNativeTest$DataArgument@5a709816
OkapiNativeTest ‑ [1] OkapiNativeTest$DataArgument@5bf0fe62
OkapiNativeTest ‑ [2] OkapiNativeTest$DataArgument@177bea38
OkapiNativeTest ‑ [2] OkapiNativeTest$DataArgument@5e4bd84a
OkapiNativeTest ‑ [2] OkapiNativeTest$DataArgument@71454b9d

OkapiNativeTest ‑ [1] OkapiNativeTest$DataArgument@1c33c17b
OkapiNativeTest ‑ [1] OkapiNativeTest$DataArgument@46dffdc3
OkapiNativeTest ‑ [1] OkapiNativeTest$DataArgument@524f3b3a
OkapiNativeTest ‑ [2] OkapiNativeTest$DataArgument@1b7c473a
OkapiNativeTest ‑ [2] OkapiNativeTest$DataArgument@2a76840c
OkapiNativeTest ‑ [2] OkapiNativeTest$DataArgument@ee86bcb

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented May 18, 2022 •
edited
Loading

Python Test Report

  12 files  ±0    12 suites  ±0   12s ⏱️ -1s
  14 tests ±0    14 ✔️ ±0  0 💤 ±0  0 ❌ ±0 
168 runs  ±0  168 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit b997ab5. ± Comparison against base commit 2abe3bc.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented May 18, 2022 •
edited
Loading

Ruby Test Report

  48 files  ±0    48 suites  ±0   3s ⏱️ ±0s
  13 tests ±0    13 ✔️ ±0  0 💤 ±0  0 ❌ ±0 
156 runs  ±0  156 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit b997ab5. ± Comparison against base commit 2abe3bc.

♻️ This comment has been updated with latest results.

@fundthmcalculus fundthmcalculus enabled auto-merge (squash) May 18, 2022 21:31
@fundthmcalculus fundthmcalculus merged commit 7a35aef into main May 18, 2022
@fundthmcalculus fundthmcalculus deleted the 424-remove-ldproofsverify_proof branch May 18, 2022 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tmarkovski tmarkovski tmarkovski approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Remove LdProofs.verify_proof
2 participants
@fundthmcalculus @tmarkovski