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

Fix grammar and remove duplicate words #544

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Vikt0rPavlik
Copy link

Changes Made

In jolt-evm-verifier/README.md:

  • Changed "packages" to "package"
  • Old: This packages implements a verifier
  • New: This package implements a verifier

In jolt-evm-verifier/src/subprotocols/HyperKZG.sol:

  • Removed duplicate word "the" in comments
  • Old: // Finally we do a MSM to get the value of the the left hand side
  • New: // Finally we do a MSM to get the value of the left hand side

Reason for Changes

  1. Grammar fix: "This package" is the correct singular form since we're referring to a single package/module. This improves readability and maintains proper English grammar.

  2. Removed duplicate word: The word "the" was accidentally duplicated in the comment. This is a simple typo fix that improves code documentation clarity.

Impact

These changes are documentation-only fixes and do not affect any functionality. They improve the readability and professionalism of the codebase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant