Skip to content
This repository has been archived by the owner on Jan 16, 2022. It is now read-only.

C4 style fixes #123

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

C4 style fixes #123

wants to merge 10 commits into from

Conversation

mikeyrf
Copy link
Contributor

@mikeyrf mikeyrf commented Dec 20, 2021

mesozoic-technology
mesozoic-technology approved these changes Jan 4, 2022
View reviewed changes
contracts/OverlayV1UniswapV3Market.sol Outdated
computeDepth(_marketLiquidity, _ovlPrice)
);

}


/// @notice Arithmetic to get depth
/// @dev Derived from cnstant product formula X*Y=K and tailored
/// @dev Derived from cnstant product formula X*Y=K and tailored
Copy link
Contributor

Choose a reason for hiding this comment

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

The typo remains.

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

@mesozoic-technology mesozoic-technology mesozoic-technology approved these changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mikeyrf @mesozoic-technology