Skip to content
This repository has been archived by the owner on Apr 21, 2021. It is now read-only.

doc: doc fix #24

Merged
merged 4 commits into from
Jan 30, 2019
Merged

doc: doc fix #24

merged 4 commits into from
Jan 30, 2019

Conversation

Berkmann18
Copy link
Member

@Berkmann18 Berkmann18 commented Jan 30, 2019

While I was translating some files to French and checking some of the source strings, I noticed some typos and some bits which missed words or didn't seem to convey the intended meaning (namely the one with "feasible" when it was meant to be the negation of it).

There's still room for improvement but that's up for a discussion.

I added the missing types and removed the uneeded `npx` call in the `commit` NPM script.
Made some bits more comprehensible and fixed some typos
@jakebolam
Copy link
Contributor

Thanks for spotting these @Berkmann18

All Contributors Kanban automation moved this from In progress to Review Jan 30, 2019
@jakebolam jakebolam merged commit 1067e44 into master Jan 30, 2019
All Contributors Kanban automation moved this from Review to Done Jan 30, 2019
@jakebolam jakebolam deleted the doc-fix branch January 30, 2019 14:39
@Berkmann18
Copy link
Member Author

@allcontributors Please add me for docs.

@allcontributors
Copy link
Contributor

@Berkmann18

I've put up a pull request to add @me! 🎉

@Berkmann18
Copy link
Member Author

@all-contributors Please add Berkmann18 for docs.

@allcontributors
Copy link
Contributor

@Berkmann18

I've put up a pull request to add @Berkmann18! 🎉

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants