Skip to content
This repository was archived by the owner on Oct 8, 2024. It is now read-only.

Upgrade to solc 0.6.12 and several improvements#32

Open
gbalabasquer wants to merge 59 commits intomasterfrom
dev
Open

Upgrade to solc 0.6.12 and several improvements#32
gbalabasquer wants to merge 59 commits intomasterfrom
dev

Conversation

@gbalabasquer
Copy link
Copy Markdown
Contributor

No description provided.

talbaneth
talbaneth previously approved these changes Sep 29, 2022
Copy link
Copy Markdown

@talbaneth talbaneth left a comment

Choose a reason for hiding this comment

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

LGTM.
We should probably still discuss the dust issue.

Copy link
Copy Markdown
Contributor

@iamchrissmith iamchrissmith left a comment

Choose a reason for hiding this comment

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

couple of suggestions that might make it a little better, but nothing blocking imo. let me know what you think of these and if you would rather just get approval

Comment thread src/DssProxyActions.sol Outdated
Comment thread src/DssProxyActions.sol Outdated
Comment thread src/DssProxyActions.sol
Comment thread src/DssProxyActions.sol
iamchrissmith
iamchrissmith previously approved these changes Nov 1, 2022
Copy link
Copy Markdown
Contributor

@iamchrissmith iamchrissmith left a comment

Choose a reason for hiding this comment

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

changes lgtm

@gbalabasquer
Copy link
Copy Markdown
Contributor Author

@talbaneth whenever you can give a final approval including the latest changes would be nice.
We would send the code as it is for auditing and if the dust issue ends up changing something else, it will be reviewed separately

talbaneth
talbaneth previously approved these changes Nov 1, 2022
Copy link
Copy Markdown

@talbaneth talbaneth left a comment

Choose a reason for hiding this comment

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

Changes lgtm, tests pass.

@gbalabasquer gbalabasquer dismissed stale reviews from talbaneth and iamchrissmith via b268061 November 21, 2022 14:17
iamchrissmith
iamchrissmith previously approved these changes Nov 22, 2022
talbaneth
talbaneth previously approved these changes Nov 23, 2022
Copy link
Copy Markdown

@talbaneth talbaneth left a comment

Choose a reason for hiding this comment

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

LGTM

@gbalabasquer gbalabasquer dismissed stale reviews from talbaneth and iamchrissmith via 40bb449 December 6, 2022 12:16
@gbalabasquer
Copy link
Copy Markdown
Contributor Author

@iamchrissmith @talbaneth will require a check in the last commit from you, thanks!

talbaneth
talbaneth previously approved these changes Dec 6, 2022
Copy link
Copy Markdown

@talbaneth talbaneth left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@iamchrissmith iamchrissmith left a comment

Choose a reason for hiding this comment

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

lgtm

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants