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

Seaport 1.3 #990

Draft
wants to merge 77 commits into
base: seaport-1.2
Choose a base branch
from
Draft

Seaport 1.3 #990

wants to merge 77 commits into from

Conversation

0age
Copy link
Contributor

@0age 0age commented Feb 14, 2023

This PR highlights changes from Seaport 1.2 to Seaport 1.3 and is meant to facilitate review and discussion of those changes.

0age and others added 30 commits February 1, 2023 15:14
deployed 1.2 across all chains
…index-overflow-test-and-refactor

add a test for index overflow with trees of different heights and refactor a bit
Bumps [http-cache-semantics](https://github.com/kornelski/http-cache-semantics) from 4.1.0 to 4.1.1.
- [Release notes](https://github.com/kornelski/http-cache-semantics/releases)
- [Commits](kornelski/http-cache-semantics@v4.1.0...v4.1.1)

---
updated-dependencies:
- dependency-name: http-cache-semantics
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
- send funds back once at the end of the fn instead of on every receive
- optimize checking seaport address
…ttp-cache-semantics-4.1.1

Bump http-cache-semantics from 4.1.0 to 4.1.1
loop through orders again if restricted/contract present
@codecov-commenter
Copy link

codecov-commenter commented Feb 14, 2023

Codecov Report

Base: 100.00% // Head: 100.00% // No change to project coverage 👍

Coverage data is based on head (2b8103a) compared to base (001a347).
Patch coverage: 100.00% of modified lines in pull request are covered.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files
@@              Coverage Diff              @@
##           seaport-1.2      #990   +/-   ##
=============================================
  Coverage       100.00%   100.00%           
=============================================
  Files               47        46    -1     
  Lines             2661      2741   +80     
  Branches           393       409   +16     
=============================================
+ Hits              2661      2741   +80     
Flag Coverage Δ
foundry 78.57% <82.00%> (+0.76%) ⬆️
production 100.00% <100.00%> (ø)
reference 100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
contracts/Seaport.sol 100.00% <ø> (ø)
contracts/lib/Consideration.sol 100.00% <ø> (ø)
contracts/lib/ConsiderationDecoder.sol 100.00% <ø> (ø)
contracts/lib/LowLevelHelpers.sol 100.00% <ø> (ø)
reference/ReferenceConsideration.sol 100.00% <ø> (ø)
contracts/lib/ConsiderationBase.sol 100.00% <100.00%> (ø)
contracts/lib/FulfillmentApplier.sol 100.00% <100.00%> (ø)
contracts/lib/OrderCombiner.sol 100.00% <100.00%> (ø)
contracts/lib/Verifiers.sol 100.00% <100.00%> (ø)
reference/lib/ReferenceConsiderationBase.sol 100.00% <100.00%> (ø)
... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@github-actions github-actions bot added the Stale label May 9, 2023
@0age 0age added Informational and removed Stale labels May 10, 2023
@ProjectOpenSea ProjectOpenSea deleted a comment from github-actions bot May 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants