lirantal / lockfile-lint

Lint an npm or yarn lockfile to analyze and detect security issues
Apache License 2.0
781 stars 35 forks source link

Update README.md #97

Closed ATGardner closed 4 years ago

ATGardner commented 4 years ago

made changes requested in PR #91 removed usage of whitelist

Just figured I can make a PR, since I saw you requesting it on Twitter right now.

Description

Replaced the bullet of the --allowed-schemes to match the updated phrasing. I also noticed there was a use of whitelist, so I replaced it with list, which seems simpler and more fitting.

Types of changes

Only changed the readme file

Related Issue

89

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Checklist:

codecov-commenter commented 4 years ago

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #97   +/-   ##
=======================================
  Coverage   99.57%   99.57%           
=======================================
  Files          11       11           
  Lines         237      237           
  Branches       41       41           
=======================================
  Hits          236      236           
  Misses          1        1           

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 b0914b6...82aa800. Read the comment docs.

lirantal commented 4 years ago

Wonderful! thanks bunch 🤗