evmts / tevm-monorepo

A library that enables next-generation UX and DX via putting an ethereum node in the browser and solidity in javascript
https://tevm.sh
MIT License
266 stars 26 forks source link

fix(precompile): remove indexed topics from log data #1482

Closed jakim929 closed 1 month ago

jakim929 commented 1 month ago

Description

Indexed values should only be included in the topics, not the data of a log

Testing

Added tests that check against expected log encoding, including one from etherscan.

changeset-bot[bot] commented 1 month ago

🦋 Changeset detected

Latest commit: d60e56d03610a48612d1ac784a98590df3ed1ee0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages | Name | Type | | ----------------- | ----- | | @tevm/precompiles | Major | | tevm | Major | | tevm-run | Major |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

vercel[bot] commented 1 month ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment | Name | Status | Preview | Updated (UTC) | | :--- | :----- | :------ | :------ | | **tevm-monorepo-tevm** | ⬜️ Ignored ([Inspect](https://vercel.com/evmts/tevm-monorepo-tevm/34pc1tn6oNUv3AmV9kZXtnSsa8Sc)) | | Oct 14, 2024 1:42am |