Skip to content

test(test-suite): update fhevm versions to 0.10.0 #1405

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
Merged
merged 1 commit into from
Dec 3, 2025
Merged

test(test-suite): update fhevm versions to 0.10.0 #1405

merged 1 commit into from
Dec 3, 2025

Conversation

@melanciani
Copy link
Contributor

@melanciani melanciani commented Nov 27, 2025

No description provided.

@melanciani melanciani requested a review from a team as a code owner November 27, 2025 19:08
@cla-bot cla-bot bot added the cla-signed label Nov 27, 2025
eudelins-zama
eudelins-zama reviewed Nov 28, 2025
View reviewed changes
Copy link
Contributor

@eudelins-zama eudelins-zama left a comment

Choose a reason for hiding this comment

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

CI is not passing on main but work on release/0.10.x...

I ran the git diff main..release/0.10.x test-suite command, and there are some differences, notably in the test-suite/e2e/test/delegateUserDecrypt/delegateUserDecryption.ts file, maybe cc @rudy-6-4

@melanciani melanciani force-pushed the melanciani/test/test-suite/main/update_fhevm_0_10_0_6 branch 2 times, most recently from e9138b8 to 7e32f67 Compare November 28, 2025 10:11
@melanciani
Copy link
Contributor Author

melanciani commented Nov 28, 2025 •
edited
Loading

yes @eudelins-zama seems like it has been modified in main recently : https://github.com/zama-ai/fhevm/pull/1092/files

but it is somewhat expected, we should consider new images if anything has been updated in main but not for 0.10, so best thing to do is keep the test-suite images introduced recently by Alex : https://github.com/zama-ai/fhevm/pull/1390/files#diff-6392bf845e5d28c9e71606db47d7889600ded5ba9775f1dfe7264d305e2a9668R41

edit: sorry, looking at it again, I think there's a mistake indeed, as the PRs I showed above are supposed to be the same as the ones included in the 0.10 branch. Let's clarify this

@melanciani melanciani force-pushed the melanciani/test/test-suite/main/update_fhevm_0_10_0_6 branch 3 times, most recently from 66ca71e to 687621d Compare December 3, 2025 13:16
@melanciani melanciani changed the title test(test-suite): update fhevm versions to 0.10.0-6 test(test-suite): update fhevm versions to 0.10.0 Dec 3, 2025
@melanciani melanciani force-pushed the melanciani/test/test-suite/main/update_fhevm_0_10_0_6 branch from 687621d to 44f2506 Compare December 3, 2025 13:18
@melanciani melanciani merged commit bb31dfa into main Dec 3, 2025
110 checks passed
@melanciani melanciani deleted the melanciani/test/test-suite/main/update_fhevm_0_10_0_6 branch December 3, 2025 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eudelins-zama eudelins-zama eudelins-zama approved these changes

Assignees

No one assigned

Labels

cla-signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@melanciani @eudelins-zama