Skip to content
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

adding smart contract changes #27

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

adding smart contract changes #27

wants to merge 2 commits into from

Conversation

sogipec
Copy link
Contributor

@sogipec sogipec commented Dec 20, 2022

No description provided.

@@ -15,7 +15,7 @@ contract OracleUSDCEURChainlink is OracleChainlinkMultiEfficient {
OracleChainlinkMultiEfficient(_stalePeriod, guardians)
{}

function _circuitChainlink() internal pure override returns (AggregatorV3Interface[2] memory) {
function circuitChainlink() public pure override returns (AggregatorV3Interface[2] memory) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

actually this need to be changed as I am doing in the borrow staked contracts. With undefined sizes

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah shit, has it been fixed in angle-solidity?

@@ -1,47 +0,0 @@
import { runTypeChain, glob } from 'typechain';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is it useless now?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahh the types generator can add it back, but since this repo essentially only has solidity code I thought I'd remove it

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