Skip to content
This repository has been archived by the owner on Nov 25, 2024. It is now read-only.

Fix: sync API incorrectly required relativePath option (fixes #178) #180

Merged
merged 1 commit into from
Jan 31, 2016
Merged

Fix: sync API incorrectly required relativePath option (fixes #178) #180

merged 1 commit into from
Jan 31, 2016

Conversation

jamesramsay
Copy link
Owner

BREAKING: CLI argument relative renamed to relativePath

@codecov-io
Copy link

Current coverage is 96.70%

Merging #180 into master will decrease coverage by -3.30% as of d046e9d

@@            master   #180   diff @@
=====================================
  Files            5      7     +2
  Stmts          114    182    +68
  Branches        17     19     +2
  Methods          0      0       
=====================================
+ Hit            114    176    +62
  Partial          0      0       
- Missed           0      6     +6

Review entire Coverage Diff as of d046e9d

Powered by Codecov. Updated on successful CI builds.

@jamesramsay jamesramsay force-pushed the optional-args branch 4 times, most recently from f5081a2 to 034f9c3 Compare January 31, 2016 01:23
Fix: sync API incorrectly required relativePath option (fixes #178)
c5818fc 
- Add tests to ensure optional args are optional.

BREAKING: CLI argument `relative` renamed to `relativePath`
jamesramsay pushed a commit that referenced this pull request Jan 31, 2016
Merge pull request #180 from jamesramsay/optional-args
489b296 
Fix: sync API incorrectly required `relativePath` option (fixes #178)
@jamesramsay jamesramsay merged commit 489b296 into master Jan 31, 2016
@jamesramsay jamesramsay deleted the optional-args branch January 31, 2016 01:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jamesramsay @codecov-io