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

chore: remove the appendix for toJSON function #65

Merged
merged 4 commits into from
Jul 3, 2024

Conversation

Ngozi-Txfusion
Copy link
Collaborator

What:

I removed the appendix for the toJSON function in our codebase.

Why:

I made this change to clean up the code by removing unnecessary parts. This improves the readability and maintainability of the toJSON function, ensuring it operates more efficiently without any redundant elements. This also resolves occasional bugs linked to the appendix, enhancing the function's stability.

@Ngozi-Txfusion Ngozi-Txfusion requested review from a team as code owners July 3, 2024 12:20
@Ngozi-Txfusion Ngozi-Txfusion changed the title chore: remove the appedix for toJSON function chore: remove the appendix for toJSON function Jul 3, 2024
Copy link

github-actions bot commented Jul 3, 2024

Visit the preview URL for this PR (updated for commit 4db7a42):

https://zksync-sdk-docs-staging--pr65-removing-appedix-for-fr5d3mfj.web.app

(expires Wed, 10 Jul 2024 12:50:03 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: f6539d831ccfa61081c09db66b409eadb94db989

@Ngozi-Txfusion Ngozi-Txfusion marked this pull request as draft July 3, 2024 12:27
@Ngozi-Txfusion Ngozi-Txfusion marked this pull request as ready for review July 3, 2024 12:32
@danijelTxFusion danijelTxFusion merged commit 8b021cd into main Jul 3, 2024
9 checks passed
@danijelTxFusion danijelTxFusion deleted the removing-appedix-for-tojson-function branch July 3, 2024 12:53
Copy link

github-actions bot commented Jul 4, 2024

🎉 This issue has been resolved in version 1.7.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

petarTxFusion pushed a commit that referenced this pull request Jul 8, 2024
* chore: remove the appedix for toJSON function

* chore: update utils

* chore: update utils

* chore: update utils.toJSON
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.

2 participants