Skip to content
This repository has been archived by the owner on May 19, 2018. It is now read-only.

Update API documentation #330

Merged
merged 1 commit into from
Feb 10, 2017
Merged

Update API documentation #330

merged 1 commit into from
Feb 10, 2017

Conversation

TimothyGu
Copy link
Contributor

@TimothyGu TimothyGu commented Jan 27, 2017

Q A
Bug fix? no
Breaking change? no
New feature? yes
Deprecations? no
Spec compliancy? no
Tests added/pass? no
Fixed tickets N/A
License MIT

The parseExpression() method was added in 898c4a7, but corresponding documentation was not. This PR remedies that.

@codecov-io
Copy link

codecov-io commented Jan 27, 2017

Current coverage is 97.65% (diff: 100%)

Merging #330 into master will not change coverage

@@             master       #330   diff @@
==========================================
  Files            20         20          
  Lines          3326       3326          
  Methods         351        351          
  Messages          0          0          
  Branches        882        882          
==========================================
  Hits           3248       3248          
  Misses           30         30          
  Partials         48         48          

Powered by Codecov. Last update 898c4a7...bc9ed98

@@ -27,6 +27,8 @@ Significant diversions are expected to occur in the future such as streaming, EB

### `babylon.parse(code, [options])`

### `babylon.parseExpression(code, [options])`
Copy link
Member

Choose a reason for hiding this comment

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

We could maybe explain what's the difference between these two methods.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@xtuc, fixed.

@danez danez merged commit 88d38a1 into babel:master Feb 10, 2017
@TimothyGu TimothyGu deleted the patch-1 branch February 10, 2017 16:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants