search

sarathsp06 / dsalgogym

Cracking The Coding Interview - Problem, Solutions and Discussions
MIT License
9 stars 6 forks source link

Index update #25

Closed arjunmahishi closed 5 years ago

arjunmahishi commented 5 years ago

Run npm run update-index to update the INDEX.md file

codecov-io commented 5 years ago

Codecov Report

Merging #25 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #25   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          14     14           
  Lines         306    306           
=====================================
  Hits          306    306
Flag Coverage Δ
#go 100% <ø> (ø) :arrow_up:
#javascript 100% <ø> (ø) :arrow_up:

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 57a95f4...13e2f44. Read the comment docs.

sarathsp06 commented 5 years ago

@arjunmahishi can you address the comments by hound ?

arjunmahishi commented 5 years ago

@sarathsp06 I think the linter is configured for avoiding console.log. But in our case, we need to log weather the file has been updated or not. Since, it's just for npm run, it should be fine.