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

Change linebreaks #20

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

Conversation

DeuceX
Copy link

@DeuceX DeuceX commented Mar 24, 2016

No description provided.

@codecov-io
Copy link

Current coverage is 100.00%

Merging #20 into master will not affect coverage as of 936887f

@@            master     #20   diff @@
======================================
  Files            7       7       
  Stmts           26      26       
  Branches         0       0       
  Methods          0       0       
======================================
  Hit             26      26       
  Partial          0       0       
  Missed           0       0       

Review entire Coverage Diff as of 936887f

Powered by Codecov. Updated on successful CI builds.

@kentcdodds
Copy link
Member

Hi @DeuceX! Thanks for giving it a try! What specifically do you need help with? If you see the Files tab here you'll see that you've added a bunch of files in the .idea directory which (as I'm sure you're aware) comes from a Jetbrains editor (like Webstorm or IntelliJ). I recommend you add this to your .gitignore_global file:

.idea/

This would prevent any Jetbrains .idea directories from being added in the future. After you've done this, you'll have to explicitly remove this from the branch on your fork using the command:

git rm .idea/

Then go ahead and commit the removal of that file and push those changes to your fork.

Once you've done that, feel free to let me know what you'd like to work on and I can help you out :-)

@DeuceX
Copy link
Author

DeuceX commented Mar 25, 2016

@kentcdodds thanks a lot!

@kentcdodds
Copy link
Member

Once you've updated your PR be sure to let me know (GitHub wont tell me when you've pushed changes to it).

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

3 participants