Skip to content

Add notice etc. to JSON output, collect errors #6 #46 - #50

Merged
steven-esser merged 11 commits into
developfrom
46-add-notice-to-json-output-file
Feb 1, 2018
Merged

steven-esser merged 11 commits into
developfrom
46-add-notice-to-json-output-file

Conversation

@johnmhoran

Copy link
Copy Markdown
Member

No description provided.

  * Create notice file, collect options selections, add to JSON output
    via cli.py.
  * Refactor version code, add to __init__.py and setup.py.
  * Modify existing tests to test changes.

Signed-off-by: John M. Horan <johnmhoran@gmail.com>
  * Add 'options' attribute to 'DeltaCode' object.
  * Fix failing tests, refactor other tests to test new structure.

Signed-off-by: John M. Horan <johnmhoran@gmail.com>
  * Consolidate custom exceptions as 'ScancodeException()'.
  * Pass informative message reflecting circumstances in which exception
    was raised.

Signed-off-by: John M. Horan <johnmhoran@gmail.com>
  * Add filename to exception message.
  * Delete redundant 'ScancodeException:' from error message.
  * Modify tests.

Signed-off-by: John M. Horan <johnmhoran@gmail.com>
  * Merged branch '6-collect-errors' into 46-add-notice-to-json-output-file to work on both at the same time.

Signed-off-by: John M. Horan <johnmhoran@gmail.com>
  * Rename custom exception.
  * Use path rather than filename in custom exception messages.
  * Simplify retrieval/generation of version info.
  * Create utility to retrieve notice info.

Signed-off-by: John M. Horan <johnmhoran@gmail.com>
johnmhoran and others added 5 commits February 1, 2018 11:45
Signed-off-by: John M. Horan <johnmhoran@gmail.com>
Signed-off-by: Steven Esser <sesser@nexb.com>
Signed-off-by: John M. Horan <johnmhoran@gmail.com>
Signed-off-by: Steven Esser <sesser@nexb.com>
Signed-off-by: Steven Esser <sesser@nexb.com>
@steven-esser
steven-esser merged commit 900cec1 into develop Feb 1, 2018
@steven-esser
steven-esser deleted the 46-add-notice-to-json-output-file branch February 1, 2018 21:27
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.

2 participants