Skip to content

Addresses the latest issue#177

Open
ktan114 wants to merge 3 commits intomasterfrom
kevin
Open

Addresses the latest issue#177
ktan114 wants to merge 3 commits intomasterfrom
kevin

Conversation

@ktan114
Copy link
Copy Markdown
Collaborator

@ktan114 ktan114 commented Sep 13, 2019

Hope it's right!

@ktan114 ktan114 requested a review from xIllusivex September 13, 2019 22:55
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #177 into master will decrease coverage by 1.58%.
The diff coverage is 4.54%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #177      +/-   ##
==========================================
- Coverage    45.5%   43.91%   -1.59%     
==========================================
  Files          22       22              
  Lines         545      567      +22     
  Branches       13       14       +1     
==========================================
+ Hits          248      249       +1     
- Misses        291      311      +20     
- Partials        6        7       +1
Impacted Files Coverage Δ
api/services/purchaseService.js 26.31% <4.54%> (-13.69%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a148e73...366d025. Read the comment docs.

table.string('exp_year', 4);
table.integer('security_number', 3);
table.boolean('active');
table.boolean('active');``
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Are those back tick's after the semi colon?

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.

3 participants