Skip to content
This repository has been archived by the owner on Feb 20, 2019. It is now read-only.

WEP: Fixed errors #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vovadyach
Copy link

Fixed errors but I could not cover all of the code.

@codecov-io
Copy link

Current coverage is 100%

Merging #31 into master will not change coverage

@@           master   #31   diff @@
===================================
  Files           8     8          
  Lines          39    39          
  Methods         0     0          
  Messages        0     0          
  Branches        0     0          
===================================
  Hits           39    39          
  Misses          0     0          
  Partials        0     0          

Powered by Codecov. Last updated by 5d1d0fe...2a740fa

@kentcdodds
Copy link
Member

Looks like you've got your branch updated. What is it that you're actually trying to accomplish now?

@mtheoryx
Copy link
Contributor

I don't really see anything added here, am I missing something?

@kentcdodds
Copy link
Member

Hey @mtheoryx,
Sorry, somehow I missed your last comment. Could you explain what you're trying to accomplish in this pull request? It looks like there are several things that need updating.

@@ -0,0 +1 @@
:::
Copy link
Member

Choose a reason for hiding this comment

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

Please remove this file

@@ -1,10 +1,7 @@
export default flatten

Copy link
Member

Choose a reason for hiding this comment

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

I'd prefer to keep the formatting how it is in this file.

@@ -1,5 +1,4 @@
export default padLeft

Copy link
Member

Choose a reason for hiding this comment

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

I'd like to keep this extra space.

@@ -1,6 +1,7 @@
import test from 'ava'
import {padLeft} from '../src'


Copy link
Member

Choose a reason for hiding this comment

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

Could you remove this extra space?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants