Skip to content

A unit test#49

Open
Jhawk1196 wants to merge 90 commits intodrsjb80:devfrom
Jhawk1196:master
Open

A unit test#49
Jhawk1196 wants to merge 90 commits intodrsjb80:devfrom
Jhawk1196:master

Conversation

@Jhawk1196
Copy link

Added test coverage support.

.coveralls.yml Outdated
@@ -0,0 +1 @@
repo_token: Sl7aI5kc1q2bGRk0Zo0GBbApBquqFlt0h
Copy link

Choose a reason for hiding this comment

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

Remove this file

.gitignore Outdated
@@ -0,0 +1,2 @@

Copy link

@sheet-t sheet-t Mar 9, 2020

Choose a reason for hiding this comment

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

Normally you shouldn't add things like blank lines into a PR but in this file you should add:

node_modules

into your .gitignore

.travis.yml Outdated
@@ -0,0 +1,19 @@
language: node_js
Copy link

Choose a reason for hiding this comment

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

Remove this file from PR

AUTHORS.md Outdated
@@ -0,0 +1,9 @@
Authors
Copy link

Choose a reason for hiding this comment

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

Remove this file as well

@sheet-t
Copy link

sheet-t commented Mar 9, 2020

Once you remove all the node_modules from this PR, post a comment and I will come check it again

@Jhawk1196
Copy link
Author

Fixed your requests

@@ -0,0 +1,12 @@
sonar.projectKey=Jhawk1196_personaswitcher
Copy link

Choose a reason for hiding this comment

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

Remove this file

@sheet-t
Copy link

sheet-t commented Mar 11, 2020

Should just be removing this one last file and you will be good

@Jhawk1196
Copy link
Author

Alright, Removed

@drsjb80 drsjb80 changed the title Pull Request from Team 2 A unit test May 15, 2020
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.

8 participants