-
Notifications
You must be signed in to change notification settings - Fork 0
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
Audit Fixes QS-3 #37
Audit Fixes QS-3 #37
Conversation
* Removed VSTOracle.sol * Removed all tests/configurations related with VST token
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.
Hey @bayou020, the brownie scripts still contain VST related data. Do we need them?
@TechnoGeek01 i don't think that we need brownie test files since we are working on foundry. @YashP16 shall we remove those files ? |
I think these brownie files are deployment files not test files. |
* Removed VST from Brownie deployement scripts
* (MasterPriceOracle.t.sol) Removed Unnecessary Comment
@@ -29,26 +28,27 @@ def main(): | |||
chainlink_oracle = ChainlinkOracle.deploy( | |||
chainlink_feeds, {"from": user}, publish_source=True | |||
) |
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.
Something wrong with formatting @bayou020.
please use Black formatter in vs-code for consistency
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.
* (deploy_oracles.py) Fixed Formatting issue.
As per Team's request on SPE-3