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

Typos fix #2329

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

Typos fix #2329

wants to merge 5 commits into from

Conversation

romashka-btc
Copy link

Fix: Corrected typos in documentation and code comments

What was changed

  1. Corrected typo algorithmical to algorithmically in CODE_OF_CONDUCT.md (line 72).
  2. Corrected typo decending to descending in core\state.go (line 399).
  3. Corrected typo panicing to panicking in db\db.go (line 111).

Why these changes were made

  • Fixing typos enhances the clarity, readability, and professionalism of the documentation and code comments.
  • Accurate text ensures better understanding and usability for developers and contributors.

Additional Notes

  • These changes focus solely on correcting typographical errors and do not affect the functionality of the code.

Copy link
Contributor

@AnkushinDaniil AnkushinDaniil left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution! Everything is good overall, please check a comment regarding changes in CODE_OF_CONDUCT.md.

CODE_OF_CONDUCT.md Outdated Show resolved Hide resolved
Copy link

codecov bot commented Dec 16, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 74.37%. Comparing base (f274e0c) to head (da32b7f).

Files with missing lines Patch % Lines
db/db.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2329      +/-   ##
==========================================
- Coverage   74.40%   74.37%   -0.03%     
==========================================
  Files         109      109              
  Lines       11493    11493              
==========================================
- Hits         8551     8548       -3     
- Misses       2277     2279       +2     
- Partials      665      666       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@romashka-btc
Copy link
Author

Thanks for your contribution! Everything is good overall, please check a comment regarding changes in CODE_OF_CONDUCT.md.

i check it. do you merge it ?

@@ -69,7 +69,7 @@ Read carefully our Contributing Guidelines to know how to contribute properly in
project. Members and maintainers must adhere to some rules regarding to pull requests
reviews and creation of issues and pull requests:

* During code reviews do not comment on coding standards and styles -focus on algorithmical,
* During code reviews do not comment on coding standards and styles -focus on algorithmically,
Copy link
Contributor

Choose a reason for hiding this comment

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

I think correct version should be:

During code reviews do not comment on coding standards and styles -focus on algorithmic,
structural or naming issues-, help to solve problem.

Copy link
Author

@romashka-btc romashka-btc Dec 16, 2024

Choose a reason for hiding this comment

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

I think correct version should be:

During code reviews do not comment on coding standards and styles -focus on algorithmic,
structural or naming issues-, help to solve problem.

yes, you are correct , but i think you can merge my version.

Copy link
Contributor

Choose a reason for hiding this comment

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

@romashka-btc please update this line with algorithmic instead of algorithmically

Copy link
Contributor

@kirugan kirugan left a comment

Choose a reason for hiding this comment

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

please check CODE_OF_CONDUCT.md suggestion

@romashka-btc
Copy link
Author

romashka-btc commented Dec 17, 2024

@AnkushinDaniil @kirugan what why you do not merge it ?

i delete old also.

@AnkushinDaniil
Copy link
Contributor

AnkushinDaniil commented Dec 18, 2024

@AnkushinDaniil @kirugan what why you do not merge it ?

Could you please make those changes @kirugan asked you to? Unfortunately we can't merge until you've finished. Thanks so much!

Copy link
Author

@romashka-btc romashka-btc left a comment

Choose a reason for hiding this comment

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

i replace it , check it pls. @AnkushinDaniil

@romashka-btc
Copy link
Author

i replace it , check it pls. @AnkushinDaniil

if i need do something more, write me.

Copy link
Contributor

Choose a reason for hiding this comment

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

There seems to be a mistake here, you have removed a whole CODE_OF_CONDUCT.md

Copy link
Author

@romashka-btc romashka-btc Dec 20, 2024

Choose a reason for hiding this comment

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

There seems to be a mistake here, you have removed a whole CODE_OF_CONDUCT.md

hmmm, and what you reccomend to do now ? @AnkushinDaniil

Copy link
Contributor

Choose a reason for hiding this comment

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

This commit appears to be correct 368fb7c
Try reverting to this commit

Copy link
Author

Choose a reason for hiding this comment

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

This commit appears to be correct 368fb7c Try reverting to this commit

ok, i will do it in 5-8 hours, pls do not close it, because i need go to work now. @AnkushinDaniil thank you !

Copy link
Author

Choose a reason for hiding this comment

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

This commit appears to be correct 368fb7c Try reverting to this commit

i can not reverting it.

Copy link
Author

Choose a reason for hiding this comment

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

This commit appears to be correct 368fb7c Try reverting to this commit

maybe you will add only another changes ?

Copy link
Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm so sorry, but I can't. Why not have a go at finding out how to do that yourself? I'm sure you'd be able to manage it, it's not that difficult.

Copy link
Author

Choose a reason for hiding this comment

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

I'm so sorry, but I can't. Why not have a go at finding out how to do that yourself? I'm sure you'd be able to manage it, it's not that difficult.

i do not understaand what i need to do. can you help me?

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.

4 participants