Skip to content
This repository has been archived by the owner on Sep 28, 2020. It is now read-only.

fix: check if contain results #300

Merged
merged 2 commits into from
Sep 26, 2019
Merged

fix: check if contain results #300

merged 2 commits into from
Sep 26, 2019

Conversation

ricardogobbosouza
Copy link
Collaborator

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

Breaking Changes

Additional Info

@codecov
Copy link

codecov bot commented Sep 26, 2019

Codecov Report

Merging #300 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #300   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           7      7           
  Lines         134    136    +2     
  Branches       28     29    +1     
=====================================
+ Hits          134    136    +2
Impacted Files Coverage Δ
src/Linter.js 100% <100%> (ø) ⬆️

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 f9b4628...5a439db. Read the comment docs.

Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Need tests or add a comment to the code when and why this happens

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants