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

[Docs] Minor Update README.rst #886

Merged
merged 2 commits into from
Aug 19, 2023

Conversation

ParthaPratimBanik
Copy link
Contributor

@ParthaPratimBanik ParthaPratimBanik commented Aug 16, 2023

Fixed minor grammatical + punctuation errors.

Description

Some grammatical + punctuation errors in README.rst:

  • the singular-plural form of the verb in the present tense
  • comma before and
  • a singular-plural form of the noun

Checklist

Fix minor grammatical + punctuation errors.
@welcome
Copy link

welcome bot commented Aug 16, 2023

Thanks for opening this pull request! We'll make sure it's perfect before merging 🤗 force
Make sure to read the contributing guide. Also, if you think that your contribution is worthy of it, you can consider adding yourself to the Contributors list (feel free to ask us if you have any doubts).

@codeclimate
Copy link

codeclimate bot commented Aug 16, 2023

Code Climate has analyzed commit 2d93a92 and detected 0 issues on this pull request.

View more on Code Climate.

@codecov-commenter
Copy link

codecov-commenter commented Aug 16, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.06% ⚠️

Comparison is base (366583e) 55.23% compared to head (17fd1e7) 55.17%.
Report is 20 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #886      +/-   ##
==========================================
- Coverage   55.23%   55.17%   -0.06%     
==========================================
  Files         297      298       +1     
  Lines       13918    13956      +38     
==========================================
+ Hits         7687     7700      +13     
- Misses       6231     6256      +25     

see 8 files with indirect coverage changes

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

@danibene danibene changed the base branch from master to dev August 17, 2023 22:19
README.rst Outdated

You have spotted an **mistake**? An **error** in a formula or code? OR there is just a step that you seems strange and you don't understand? **Please let us know!** We are Human beings, and we'll appreciate any inquiry.
You have spotted a **mistake**? An **error** in a formula or code? OR there is just a step that you seem strange and you don't understand? **Please let us know!** We are Human beings, and we'll appreciate any inquiry.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
You have spotted a **mistake**? An **error** in a formula or code? OR there is just a step that you seem strange and you don't understand? **Please let us know!** We are Human beings, and we'll appreciate any inquiry.
You have spotted a **mistake**? An **error** in a formula or code? OR there is just a step that seems strange and you don't understand? **Please let us know!** We are human beings, and we'll appreciate any inquiry.

Copy link
Collaborator

@danibene danibene left a comment

Choose a reason for hiding this comment

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

Thank you for this :) I left one suggested change, otherwise looks good to me!

suggested changes are done!
@ParthaPratimBanik
Copy link
Contributor Author

Thank you for this :) I left one suggested change, otherwise looks good to me!

Thanks for your suggestion. It is done!

@danibene danibene merged commit ed5e999 into neuropsychology:dev Aug 19, 2023
9 checks passed
@welcome
Copy link

welcome bot commented Aug 19, 2023

landing
Congrats on merging your first pull request! 🎉🍾 We're looking forward to your next one!

@ParthaPratimBanik ParthaPratimBanik deleted the readme_fix branch August 20, 2023 00:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants