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

Move random_test_initializer to separate file [SyncWith: crypto3-zk#328] #328

Merged
merged 1 commit into from
May 12, 2024

Conversation

x-mass
Copy link
Member

@x-mass x-mass commented May 8, 2024

No description provided.

@x-mass x-mass changed the title Move random_test_initializer to separate file Move random_test_initializer to separate file [SyncWith: crypto3-zk#328] May 8, 2024
@nil-foundation-cicd-bot
Copy link

nil-foundation-cicd-bot bot commented May 8, 2024

PRs, synced with the current issue:

Note: Please do not edit this comment; it's generated automatically.

@nil-foundation-cicd-bot
Copy link

Tests will be run in sync with other PRs containing [SyncWith NilFoundation/crypto3-zk#328] in title. You can find related PRs as linked with mentioned issue.

Copy link

github-actions bot commented May 8, 2024

Test Results

  30 files    30 suites   58s ⏱️
  79 tests   79 ✔️ 0 💤 0
168 runs  168 ✔️ 0 💤 0

Results for commit 5d47b83.

♻️ This comment has been updated with latest results.

@x-mass x-mass force-pushed the move-test-initializer branch from 99382c7 to 2c082fd Compare May 8, 2024 15:26
@x-mass x-mass requested a review from martun May 8, 2024 17:19
@x-mass x-mass marked this pull request as ready for review May 8, 2024 17:19
@@ -178,7 +84,6 @@ target_link_libraries(${CMAKE_WORKSPACE_NAME}_${CURRENT_PROJECT_NAME} INTERFACE
${CMAKE_WORKSPACE_NAME}::hash
${CMAKE_WORKSPACE_NAME}::multiprecision
${CMAKE_WORKSPACE_NAME}::containers)
# marshalling::crypto3_zk)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We probably have this dependency, but we choose not to list it here, that's why we had this comment. But it's ok to delete it.

@x-mass x-mass force-pushed the move-test-initializer branch from 2c082fd to 5d47b83 Compare May 10, 2024 17:34
@x-mass x-mass merged commit 194e46f into master May 12, 2024
8 checks passed
@x-mass x-mass deleted the move-test-initializer branch May 12, 2024 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants