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 #6975

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

fix multiple typos #6975

wants to merge 5 commits into from

Conversation

crStiv
Copy link

@crStiv crStiv commented Dec 20, 2024

Description

This pull request addresses a series of minor typos and formatting issues across various scripts and configuration files in the repository. These changes ensure better readability, accuracy, and adherence to coding standards. The typos corrected include:

  • Fixed spelling mistakes in comments (e.g., "formated" -> "formatted").
  • Corrected inconsistencies in variable names and comments to improve clarity (e.g., "polakdot" -> "polkadot").
  • Adjusted misspelled words and clarified usage descriptions in workflows.

Integration

The changes are self-contained and do not require specific integration steps. All updates are related to documentation, comments, and non-functional text changes within scripts and workflows. No runtime behavior or logic is affected, and downstream projects can safely adopt these updates without risk of regression.

Review Notes

The review should focus on verifying that:

  1. All typos and formatting errors have been correctly addressed.
  2. No unintended changes to code logic or functionality have been introduced.
  3. Comments and descriptions accurately reflect the intended behavior and are clear to future maintainers.

Below are some examples of the changes:

- # Retun list of the runtimes filterd
+ # Retun list of the runtimes filtered

- # Displays formated results of the git log command
+ # Displays formatted results of the git log command

- description: Current final release tag in the format polakdot-stableYYMM
+ description: Current final release tag in the format polkadot-stableYYMM


# Checklist

* [ ] My PR includes a detailed description as outlined in the "Description" and its two subsections above.
* [ ] My PR follows the [labeling requirements](
https://github.com/paritytech/polkadot-sdk/blob/master/docs/contributor/CONTRIBUTING.md#Process
) of this project (at minimum one label for `T` required)
    * External contributors: ask maintainers to put the right label on your PR.
* [ ] I have made corresponding changes to the documentation (if applicable)
* [ ] I have added tests that prove my fix is effective or that my feature works (if applicable)

You can remove the "Checklist" section once all have been checked. Thank you for your contribution!

✄ -----------------------------------------------------------------------------

@crStiv crStiv requested review from a team as code owners December 20, 2024 21:11
@cla-bot-2021
Copy link

cla-bot-2021 bot commented Dec 20, 2024

User @crStiv, please sign the CLA here.

@crStiv crStiv changed the title Fix fix multiple typos Dec 20, 2024
@bkchr bkchr added the R0-silent Changes should not be mentioned in any release notes label Dec 20, 2024
@crStiv
Copy link
Author

crStiv commented Dec 20, 2024

@bkchr Cool, I'm glad my changes helped you, and I'll be much obliged if this PR will become merged, tnx for your review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants