Skip to content

Spellcheck docs#8548

Merged
fjy merged 2 commits intoapache:masterfrom
ccaominh:docs-spellcheck
Sep 17, 2019
Merged

Spellcheck docs#8548
fjy merged 2 commits intoapache:masterfrom
ccaominh:docs-spellcheck

Conversation

@ccaominh
Copy link
Copy Markdown
Contributor

Description

Fix spelling mistakes in docs and add CI job for running spellcheck on docs.


This PR has:

  • been self-reviewed.
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.

Fix spelling mistakes in docs and add CI job for running spellcheck on
docs.
@vogievetsky
Copy link
Copy Markdown
Contributor

vogievetsky commented Sep 17, 2019

This. Is. Amazing. 👍

@fjy fjy merged commit 7dcbaca into apache:master Sep 17, 2019
@ccaominh ccaominh deleted the docs-spellcheck branch September 17, 2019 19:51
clintropolis pushed a commit to clintropolis/druid that referenced this pull request Sep 23, 2019
* Spellcheck docs

Fix spelling mistakes in docs and add CI job for running spellcheck on
docs.

* Add missing license header
@clintropolis clintropolis added this to the 0.16.0 milestone Sep 23, 2019
fjy pushed a commit that referenced this pull request Sep 23, 2019
* Spellcheck docs

Fix spelling mistakes in docs and add CI job for running spellcheck on
docs.

* Add missing license header
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants