Skip to content

docs: fix NatSpec in simulateAndRevert#936

Merged
mmv08 merged 1 commit intosafe-fndn:mainfrom
PaulRBerg:docs/simulate-and-revert
Apr 10, 2025
Merged

docs: fix NatSpec in simulateAndRevert#936
mmv08 merged 1 commit intosafe-fndn:mainfrom
PaulRBerg:docs/simulate-and-revert

Conversation

@PaulRBerg
Copy link
Copy Markdown
Contributor

@PaulRBerg PaulRBerg commented Apr 4, 2025

This PR fixes the NatSpec documentation in simulateAndRevert such that it no longer incorrectly claims that the return data is ABI encoded (bool, bytes), but instead a custom packed encoding.

@PaulRBerg PaulRBerg requested a review from a team as a code owner April 4, 2025 14:22
@PaulRBerg PaulRBerg requested review from akshay-ap, nlordell, remedcu and rmeissner and removed request for a team April 4, 2025 14:22
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 4, 2025

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@PaulRBerg
Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

github-actions Bot added a commit that referenced this pull request Apr 4, 2025
@nlordell
Copy link
Copy Markdown
Collaborator

nlordell commented Apr 4, 2025

Hmm, looks like some zkSync tests are failing unrelated to your PR.

@nlordell
Copy link
Copy Markdown
Collaborator

@PaulRBerg - can you rebase your PR? @mmv08 fixed the zkSync tests on main so CI should pass now.

@PaulRBerg PaulRBerg force-pushed the docs/simulate-and-revert branch from c8c7c02 to 94d5a82 Compare April 10, 2025 11:00
@PaulRBerg
Copy link
Copy Markdown
Contributor Author

Done @nlordell

@mmv08 mmv08 merged commit d89d156 into safe-fndn:main Apr 10, 2025
9 of 23 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 10, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants