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 spelling errors #604

Closed
wants to merge 7 commits into from
Closed

Fix spelling errors #604

wants to merge 7 commits into from

Conversation

Pronoss
Copy link

@Pronoss Pronoss commented Jan 14, 2025

  1. substractions -> subtractions
  2. Formated -> Formatted
  3. aligment -> alignment
  4. ceate -> create
  5. elemts -> elements
  6. colums -> columns

Motivation:
Fixing these spelling errors improves the clarity and professionalism of the codebase.

Checklist:

  • Performed self-review of the code.
  • Commented changes in complex or ambiguous areas.
  • Ensured unit tests pass locally.
  • Verified changes align with repository contribution guidelines.

@gbotrel gbotrel added the typo Typo fixes label Jan 14, 2025
@ivokub
Copy link
Collaborator

ivokub commented Jan 20, 2025

Thanks for the contribution. I'm going to have it close though - first the tensor commitment package is removed, secondly one of the typo fixes breaks compilation gopter.NewFormatedReporter -> gopter.NewFormattedReporter. Fixing those it would leave only two very minor comments which are code generated. So it is not worth it.

@ivokub ivokub closed this Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
typo Typo fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants