Skip to content

Use Promise Spec Terminology Where Appropriate #38

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

Merged
merged 1 commit into from
Mar 29, 2019
Merged

Use Promise Spec Terminology Where Appropriate #38

merged 1 commit into from
Mar 29, 2019

Conversation

dhurlburtusa
Copy link
Contributor

This change is to help the documentation to fall more inline with the v6.x Promise spec terminology.

Use "Fulfilled" in place of "resolved".
Use "Pending" in place of "loading" when "loading" didn't feel more natural.
Capitalize "O" in object when used as a datatype to use capitalization preferred by JSDoc and other tools. Only boolean, number, and string are expected to be all lowercase (assuming you are expecting the primitives and not the wrapper objects).
Fix a few minor mistakes (grammar, unescaped tag syntax, etc).

Use "Fulfilled" in place of "resolved".
Use "Pending" in place of "loading" when "loading" didn't feel more natural.
Capitalize "O" in object when used as a datatype to use capitalization preferred by JSDoc and other tools. Only `boolean`, `number`, and `string` are expected to be all lowercase (assuming you are expecting the primitives and not the wrapper objects).
Fix a few minor mistakes (grammar, unescaped tag syntax, etc).
@codecov
Copy link

codecov bot commented Mar 29, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #38   +/-   ##
=======================================
  Coverage   99.61%   99.61%           
=======================================
  Files           5        5           
  Lines         525      525           
  Branches      112      112           
=======================================
  Hits          523      523           
  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 23c9d8d...7d95c01. Read the comment docs.

Copy link
Member

@ghengeveld ghengeveld left a comment

Choose a reason for hiding this comment

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

Excellent improvements, thanks for the sharp eye.

@ghengeveld ghengeveld merged commit 94ec322 into async-library:master Mar 29, 2019
@dhurlburtusa dhurlburtusa deleted the patch-1 branch March 29, 2019 21:45
@ghengeveld
Copy link
Member

@all-contributors please add @dhurlburtusa for doc

@allcontributors
Copy link
Contributor

@ghengeveld

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

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.

2 participants