-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add audited contracts #10
Conversation
Signed-off-by: Matt Rice <matthewcrice32@gmail.com>
@@ -0,0 +1,46 @@ | |||
// SPDX-License-Identifier: AGPL-3.0-only |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we want mixing licenses? this repo currently has BUSL-1.1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Reinis-FRP you're right. it should be the same license that this repo currently has.
/** | ||
* @notice ChronicleMedianDestinationAdapter contract to expose Oval data via the standard Chronicle interface. | ||
*/ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: forge fmt
would remove empty line here and everywhere else after the contract NatSpec
Signed-off-by: chrismaree <christopher.maree@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! importantly, we are only adding new adapters & tests and all the base Oval logic goes un-touched.
Co-authored-by: chrismaree <christopher.maree@gmail.com>
This adds previously audited contracts to this repository.
Note: to move these over, I had to do a find/replace OVAL -> Oval. After that, the compilation seemed to work, but there may be other issues. If you notice something off, let me know and I can fix.