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 documentation files #4328

Merged
merged 4 commits into from
Nov 25, 2024
Merged

Conversation

vtjl10
Copy link
Contributor

@vtjl10 vtjl10 commented Nov 15, 2024

This pull request contains changes to improve clarity, correctness and structure.

Description correction:
Corrected "Minor changes changes" to "Minor changes".
Corrected "piplelines" to "pipelines".
Corrected "gossmer" to "gossamer".
Corrected "Sate" to "State".

Please review the changes and let me know if any additional changes are needed.

@vtjl10 vtjl10 requested review from P1sar and timwu20 as code owners November 15, 2024 17:48
@CLAassistant
Copy link

CLAassistant commented Nov 15, 2024

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@EclesioMeloJunior EclesioMeloJunior left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thks for your contribution!

@vtjl10
Copy link
Contributor Author

vtjl10 commented Nov 19, 2024

@EclesioMeloJunior Do I need to make any additional changes before merging?

@dimartiro
Copy link
Contributor

@vtjl10 thanks for fixing this typos!
Could you please rebase your branch, it is out-of-date with our develop.
After that we will be able to merge ❤️

@dimartiro dimartiro added P-low this is mostly nice to have. C-simple Minor changes changes, no additional research needed. Good first issue/review. S-doc documentation related. labels Nov 21, 2024
Copy link

codecov bot commented Nov 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 53.12%. Comparing base (d1ca7aa) to head (2735c5a).
Report is 203 commits behind head on development.

Additional details and impacted files
@@               Coverage Diff               @@
##           development    #4328      +/-   ##
===============================================
+ Coverage        50.51%   53.12%   +2.60%     
===============================================
  Files              230      299      +69     
  Lines            29006    38252    +9246     
===============================================
+ Hits             14653    20322    +5669     
- Misses           12856    15996    +3140     
- Partials          1497     1934     +437     
---- 🚨 Try these New Features:

@haikoschol haikoschol enabled auto-merge (squash) November 25, 2024 15:05
Copy link
Contributor

@haikoschol haikoschol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tyvm @vtjl10!

@haikoschol haikoschol merged commit 2f6d82b into ChainSafe:development Nov 25, 2024
24 checks passed
dimartiro pushed a commit that referenced this pull request Nov 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-simple Minor changes changes, no additional research needed. Good first issue/review. P-low this is mostly nice to have. S-doc documentation related.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants