Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed issues #70 and #71 #86

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

cryolune
Copy link
Member

@codecov-io
Copy link

codecov-io commented Mar 31, 2021

Codecov Report

Merging #86 (55afbca) into master (16da1c2) will not change coverage.
The diff coverage is n/a.

❗ Current head 55afbca differs from pull request most recent head 9cb6ef7. Consider uploading reports for the commit 9cb6ef7 to get more accurate results
Impacted file tree graph

@@          Coverage Diff           @@
##           master     #86   +/-   ##
======================================
  Coverage    6.53%   6.53%           
======================================
  Files          20      20           
  Lines         597     597           
  Branches       68      68           
======================================
  Hits           39      39           
  Misses        506     506           
  Partials       52      52           

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 16da1c2...9cb6ef7. Read the comment docs.

@dlebauer dlebauer requested a review from im-prakher March 31, 2021 16:27
@cryolune cryolune force-pushed the theakhiljha-coverage branch from 17d4136 to 0b01048 Compare April 5, 2021 21:08
@cryolune cryolune force-pushed the theakhiljha-coverage branch from 0838530 to 43bcff6 Compare April 5, 2021 22:07
@dlebauer
Copy link
Member

dlebauer commented Apr 8, 2021

Could you check that the linter is implemented correctly?

@im-prakher im-prakher mentioned this pull request Apr 11, 2021
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