Ability to ignore packages #46

jvorcak · 2019-12-17T13:50:33Z

Fixes #41

You can now optionally specify this in your config

"ignored_packages": {
    "python_pipenv": [
      "PyInquirer",
      "setuptools"
    ],
    "npm": [],
    "maven": [],
    "yarn": []
  }

4rokis

I would love to see at least one test there

license_sh/helpers.py

tests/runners/test_helpers.py

license-sh

jvorcak · 2019-12-21T13:30:37Z

I've added test cases. This is quite a simple functionality so I couldn't think of more corner cases.

Ability to ignore packages

485499d

jvorcak mentioned this pull request Dec 17, 2019

Document ignore_packages option #47

Closed

jvorcak requested review from jankoritak and 4rokis December 17, 2019 13:52

jankoritak approved these changes Dec 17, 2019

View reviewed changes

4rokis suggested changes Dec 17, 2019

View reviewed changes

license_sh/helpers.py Show resolved Hide resolved

tests/runners/test_helpers.py Show resolved Hide resolved

license-sh Show resolved Hide resolved

license-sh Show resolved Hide resolved

license-sh Show resolved Hide resolved

jvorcak mentioned this pull request Dec 17, 2019

Add __main__ #49

Closed

Merge branch 'master' into feature/ignore-packages

be8845d

jvorcak merged commit b831f0b into master Dec 21, 2019

jvorcak deleted the feature/ignore-packages branch December 21, 2019 13:30

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Ability to ignore packages #46

Ability to ignore packages #46

jvorcak commented Dec 17, 2019 •

edited

4rokis left a comment

jvorcak commented Dec 21, 2019

Ability to ignore packages #46

Ability to ignore packages #46

Conversation

jvorcak commented Dec 17, 2019 • edited

4rokis left a comment

Choose a reason for hiding this comment

jvorcak commented Dec 21, 2019

jvorcak commented Dec 17, 2019 •

edited