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: multiple typos of different importance #3117

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

DeVikingMark
Copy link

Description

This pull request addresses various typos across multiple files. The corrections include changes to spelling, grammar, and phrasing to improve clarity and consistency throughout the codebase and documentation.

What does it do?

  • Fixes typos in comments and documentation across several files.
  • Corrects grammatical errors to enhance readability and accuracy.
  • Refines phrasing to align with best practices for technical writing.

What important points reviewers should know?

  • Changes are limited to non-functional aspects of the code (e.g., comments and documentation). No functionality or logic was altered.
  • This is a minor but wide-reaching improvement focused on maintaining code quality.

Is there something left for follow-up PRs?

No additional work related to these changes is anticipated. This PR comprehensively addresses the identified typos.

What alternative implementations were considered?

No alternatives were necessary, as this PR strictly focuses on typo corrections and grammatical improvements.

Are there relevant PRs or issues in other repositories (Substrate, Polkadot, Frontier, Cumulus)?

None are directly related to these changes. The fixes are isolated to this repository.

What value does it bring to the blockchain users?

Although the changes are not directly user-facing, they improve codebase maintainability, readability, and developer experience, reducing potential confusion for contributors and maintainers.

Allow edits by maintainers

✔️ Checked.

@RomarQ
Copy link
Contributor

RomarQ commented Dec 21, 2024

Airdrop farming, but a useful contribution.

@RomarQ RomarQ added I5-documentation 📄 Documentation needs fixing, improving or augmenting. B0-silent Changes should not be mentioned in any release notes D2-notlive PR doesn't change runtime code (so can't be audited) labels Dec 21, 2024
@DeVikingMark
Copy link
Author

Airdrop farming, but a useful contribution.

then why not merging :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B0-silent Changes should not be mentioned in any release notes D2-notlive PR doesn't change runtime code (so can't be audited) I5-documentation 📄 Documentation needs fixing, improving or augmenting. suspicious-contribution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants