Skip to content

Conversation

@0xpanoramix
Copy link
Contributor

Description

This PR adds a logger which prints the reports accumulated during the scans.
The metadata object is no longer passed to the scanners, but is kept in case of future use.

Changes include

  • Bugfix (non-breaking change that solves an issue).
  • Hotfix (change that solves an urgent issue, and requires immediate attention).
  • New feature (non-breaking change that adds functionality).
  • Breaking change (change that is not backwards-compatible and/or changes current functionality).
  • Documentation update (change added to any kind of documentation).

Checklist

  • I have assigned this PR to myself.
  • I have added at least 1 reviewer.
  • I have added the relevant labels.
  • I have updated the official documentation.
  • I have added sufficient documentation in code.

Testing

  • I have tested this code with the official test suite.
  • I have tested this code manually.

Manual tests

Ran against the SimpleContract.sol.

Signed-off-by: Luca Georges Francois <luca@quartz.technology>
Signed-off-by: Luca Georges Francois <luca@quartz.technology>
@0xpanoramix 0xpanoramix added the enhancement New feature or request label Aug 1, 2023
@0xpanoramix 0xpanoramix self-assigned this Aug 1, 2023
@0xpanoramix 0xpanoramix merged commit b70d082 into main Aug 1, 2023
@0xpanoramix 0xpanoramix deleted the feat/add-report-logger branch August 1, 2023 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants