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

eddsa poseidon update #2079

Merged
merged 4 commits into from
Dec 10, 2024
Merged

eddsa poseidon update #2079

merged 4 commits into from
Dec 10, 2024

Conversation

artwyman
Copy link
Member

This steals the build fix from #2076 to make unit tests work. Thanks @robknight.

Attn: @cedoor regarding the packaging issue we're working around. If you grab this branch and remove the build config change, you'll see the error. Probably something zk-kit needs to either fix, or document for other devs.

@artwyman artwyman requested a review from robknight October 31, 2024 02:02
@artwyman artwyman self-assigned this Oct 31, 2024
@cedoor
Copy link
Contributor

cedoor commented Oct 31, 2024

Thanks @artwyman

@robknight is this an issue with vitest specifically then? I wonder if that happens with other JS testing (or general) frameworks (not with Jest apparently).

@robknight
Copy link
Member

robknight commented Oct 31, 2024

@cedoor

Thanks @artwyman

@robknight is this an issue with vitest specifically then? I wonder if that happens with other JS testing (or general) frameworks (not with Jest apparently).

I'm not sure. I am trying to avoid CommonJS-only packages where possible, as they seem to have more compatibility issues, but I don't think that there's anything that you can do about this particular issue.

@artwyman
Copy link
Member Author

This is ready to go, but I'm holding it back until after Devcon to minimize risk.

# Conflicts:
#	packages/lib/pod/package.json
#	packages/pcd/eddsa-pcd/package.json
#	yarn.lock
@artwyman artwyman added this pull request to the merge queue Dec 10, 2024
Merged via the queue into main with commit 11a316d Dec 10, 2024
1 check passed
@artwyman artwyman deleted the artwyman/eddsa-poseidon-update branch December 10, 2024 02:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants