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 typos in expression.rs and verify.rs #1383

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

teenager-ETH
Copy link

Pull Request Template

Title: Fix typos in expression.rs and verify.rs


Description

This PR fixes minor typos in comments and output messages within the following files:

  1. zokrates_ast/src/ir/expression.rs

    • Corrected the word "reduceable" to "reducible" in comments.
  2. zokrates_cli/src/ops/nova/verify.rs

    • Fixed a typo in a user-facing message:
      Changed "succesfully" to "successfully."

Changes

Files Modified

  1. zokrates_ast/src/ir/expression.rs

    • Line 195:
      Before: it is reduceable to 0
      After: it is reducible to 0
    • Line 223:
      Before: it is not reduceable to a summand
      After: it is not reducible to a summand
  2. zokrates_cli/src/ops/nova/verify.rs

    • Line 116:
      Before: Compressed proof succesfully verified
      After: Compressed proof successfully verified

Motivation and Context

Fixing these typos improves code clarity and ensures polished user-facing messages. Correct spelling and grammar contribute to better code documentation and a professional user experience.


How Has This Been Tested?

  • Verified the changes compile successfully.
  • Ran basic checks to ensure no unintended behaviors were introduced.

Checklist

  • My changes follow the project's coding style and guidelines.
  • I have performed a self-review of my code.
  • I have commented on changes where necessary.
  • I have ensured the code compiles without errors.

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