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.

Sign up for GitHub

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

Jump to bottom

Fix typo #2862

Merged
merged 3 commits into from
Dec 31, 2019
Merged

Fix typo #2862

merged 3 commits into from
Dec 31, 2019

Conversation

e-kwsm
Copy link
Contributor

@e-kwsm e-kwsm commented Dec 24, 2019

No description provided.

greglandrum
greglandrum requested changes Dec 27, 2019
View reviewed changes
Copy link
Member

@greglandrum greglandrum left a comment

Choose a reason for hiding this comment

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

A few minor corrections/tweaks added here.

Code/GraphMol/Aromaticity.cpp Outdated Show resolved Hide resolved
Code/GraphMol/Aromaticity.cpp Outdated Show resolved Hide resolved
Code/GraphMol/Depictor/DepictUtils.cpp Outdated Show resolved Hide resolved
Code/GraphMol/Depictor/EmbeddedFrag.cpp Outdated Show resolved Hide resolved
Code/GraphMol/Depictor/RDDepictor.cpp Outdated Show resolved Hide resolved
Code/GraphMol/Depictor/RDDepictor.cpp Outdated Show resolved Hide resolved
Code/ML/InfoTheory/InfoBitRanker.h Outdated Show resolved Hide resolved
Docs/Book/RDKit_Book.rst Outdated Show resolved Hide resolved
@greglandrum
Copy link
Member

greglandrum commented Dec 27, 2019 •
edited

@e-kwsm : thanks for doing this!
It looks like you must have run a spelling/grammar checker across the code; which tool did you use for that?

@e-kwsm
Copy link
Contributor Author

e-kwsm commented Dec 28, 2019

It looks like you must have run a spelling/grammar checker across the code; which tool did you use for that?

I used aspell and typochecker.

@greglandrum greglandrum added this to the 2020_03_1 milestone Dec 31, 2019
@greglandrum greglandrum merged commit 5cd27a2 into rdkit:master Dec 31, 2019
@e-kwsm e-kwsm deleted the fix-typo branch December 31, 2019 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@greglandrum greglandrum greglandrum approved these changes

Assignees
No one assigned
Labels
documentation enhancement
Projects
None yet
Milestone
2020_03_1
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@e-kwsm @greglandrum