Skip to content

Conversation

slowy07
Copy link
Contributor

@slowy07 slowy07 commented Aug 30, 2021

fixing typo spelling grammar and replace to correct words with reference from merriam webster

  • User visible changes (including notable bug fixes) are documented in whats-new.rst
  • Fix typo

Copy link
Collaborator

@max-sixty max-sixty 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 very much @slowy07 !

@max-sixty max-sixty merged commit 588cc62 into pydata:main Aug 30, 2021
@slowy07
Copy link
Contributor Author

slowy07 commented Aug 30, 2021

thank you sir @max-sixty

@github-actions
Copy link
Contributor

Unit Test Results

         6 files  ±0           6 suites  ±0   52m 37s ⏱️ ±0s
16 225 tests ±0  14 490 ✔️ ±0  1 735 💤 ±0  0 ❌ ±0 
90 546 runs  ±0  82 369 ✔️ ±0  8 177 💤 ±0  0 ❌ ±0 

Results for commit 588cc62. ± Comparison against base commit 588cc62.

can be used externally to meet domain-specific needs.

We can generalize the community's needs into three main catagories:
We can generalize the community's needs into three main caeagories:
Copy link
Collaborator

Choose a reason for hiding this comment

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

shouldn't this be

Suggested change
We can generalize the community's needs into three main caeagories:
We can generalize the community's needs into three main categories:

?

Since this PR has already been merged we would need a new PR. @slowy07, would you be up for that?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

okay sir @keewis ,i'll do it,thanks for the review

@slowy07 slowy07 deleted the minor-changes branch August 30, 2021 04:57
@slowy07 slowy07 mentioned this pull request Aug 30, 2021
2 tasks
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.

3 participants