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

Chore: Fix spellings and update typescript compiler version to 3.X #3356

Merged
1 commit merged into from Dec 31, 2018
Merged

Chore: Fix spellings and update typescript compiler version to 3.X #3356

1 commit merged into from Dec 31, 2018

Conversation

ghost
Copy link

@ghost ghost commented Dec 30, 2018

  1. For "Sails" in both Chinese and English versions, we should also add the links for them.
  2. Union the spellings of "Plugin" instead of "Plug-in".

3. IMPORTANT!!
Update to the latest version of Typescript, because yard has used "unknown" type, which MUST BE supported since v3.X.

Here's the screenshot before update:
unknown

  • npm test passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

@ghost ghost requested review from atian25, killagu and SunShinewyf December 30, 2018 10:12
@codecov
Copy link

codecov bot commented Dec 30, 2018

Codecov Report

Merging #3356 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3356   +/-   ##
=======================================
  Coverage   99.76%   99.76%           
=======================================
  Files          29       29           
  Lines         844      844           
=======================================
  Hits          842      842           
  Misses          2        2

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cd52b06...463866e. Read the comment docs.

@ghost ghost changed the title Chore: Add missing links for "Sails" and union the spellings of "Plugin" Chore: Fix spellings and update typescript compiler version to 3.X Dec 30, 2018
@ghost ghost requested a review from whxaxes December 31, 2018 00:36
1) For "Sails" in both Chinese and English versions, we should also
add the links for them.
2) Union the spellings of "Plugin" instead of "Plug-in".
@ghost ghost merged commit ac0f13b into eggjs:master Dec 31, 2018
@ghost ghost deleted the FixPlugIn branch December 31, 2018 04:00
popomore pushed a commit that referenced this pull request Dec 31, 2018
Chore: Add missing links for "Sails" and union the spellings of "Plugin" (#3356)

1. For "Sails" in both Chinese and English versions, we should also add the links for them.
2. Union the spellings of "Plugin" instead of "Plug-in".
3. IMPORTANT:
Update to the latest version of Typescript, because yard has used "unknown" type, which MUST BE supported since v3.X.
This pull request was closed.
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.

None yet

1 participant