Skip to content

chore: merge v0.2.0-rc.0 to main #669

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 7 commits into from
Closed

chore: merge v0.2.0-rc.0 to main #669

wants to merge 7 commits into from

Conversation

bidzyyys
Copy link
Collaborator

No description provided.

bidzyyys and others added 7 commits April 22, 2025 16:18
@bidzyyys
build: openzeppelin-stylus-proc v0.2.0 (#631)
6226a16
@bidzyyys
build(ci): improve CI jobs (#637)
66a0a81
@bidzyyys @0xNeshi @dependabot
fix: export-abi feature (#602)
16cc1ff 
Resolves #439 

@0xNeshi the idea is to have a working solution within the current
Stylus SDK's limitations.
We can work on a "clean" solution after the audit starts.

#### PR Checklist

<!--
Before merging the pull request all of the following must be completed.
Feel free to submit a PR or Draft PR even if some items are pending.
Some of the items may not apply.
-->

- [x] Tests
- [x] Documentation
- [x] Changelog

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: 0xNeshi <nenad.misic@openzeppelin.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@bidzyyys
build: update nitro-testnode (#648)
c9f4935
@bidzyyys @qalisander @0xNeshi
feat: add fast Pedersen hash (Starknet version) (#644)
2350766 
Starknet version of pedersen hash function. 
Shift operations for bigintegers.

Resolves #269 and #516

#### PR Checklist
- [x] Tests
- [x] Documentation
- [x] Changelog

---------

Co-authored-by: Alisander Qoshqosh <37006439+qalisander@users.noreply.github.com>
Co-authored-by: Alisander Qoshqosh <qalisander@gmail.com>
Co-authored-by: Nenad <nenad.misic@openzeppelin.com>
@0xNeshi @bidzyyys
@dependabot @qalisander @YanVictorSN @nickventon
feat: Constructors and the new inheritance model from stylus-sdk@0.9 (#…
788c7b2 
…639)

Adds constructors (drops koba tooling) and new inheritance model from stylus-sdk@0.9

Resolves #492 , resolves #439

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Daniel Bigos <daniel.bigos@icloud.com>
Co-authored-by: Daniel Bigos <daniel.bigos@openzeppelin.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Alisander Qoshqosh <37006439+qalisander@users.noreply.github.com>
Co-authored-by: Yan Victor SN <98413246+YanVictorSN@users.noreply.github.com>
Co-authored-by: Nicholas Kelson <ambala.sweets.dev@gmail.com>
@bidzyyys
build: bump to v0.2.0-rc.0 (#630)
5ed33dd
@netlify Netlify
Copy link

netlify bot commented May 22, 2025 •
edited
Loading

✅ Deploy Preview for contracts-stylus ready!

Name Link
🔨 Latest commit 5ed33dd
🔍 Latest deploy log https://app.netlify.com/projects/contracts-stylus/deploys/682f41bbb547d300083e02ca
😎 Deploy Preview https://deploy-preview-669--contracts-stylus.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@bidzyyys bidzyyys changed the title chore: merge v0.2.0-rc.0 chore: merge v0.2.0-rc.0 to main May 22, 2025
@codecov Codecov
Copy link

codecov bot commented May 22, 2025

Codecov Report

Attention: Patch coverage is 54.54545% with 385 lines in your changes missing coverage. Please review.

Project coverage is 85.5%. Comparing base (a67ab70) to head (5ed33dd).
Report is 4 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
contracts/src/token/erc20/extensions/erc4626.rs 4.7% 140 Missing ⚠️
contracts/src/token/erc721/extensions/wrapper.rs 39.6% 73 Missing ⚠️
contracts/src/finance/vesting_wallet.rs 30.0% 35 Missing ⚠️
contracts/src/token/erc20/extensions/permit.rs 0.0% 28 Missing ⚠️
contracts/src/token/erc20/extensions/wrapper.rs 63.6% 20 Missing ⚠️
contracts/src/token/erc20/extensions/flash_mint.rs 50.0% 17 Missing ⚠️
...ntracts/src/token/erc721/extensions/consecutive.rs 75.0% 15 Missing ⚠️
contracts/src/access/ownable.rs 62.5% 9 Missing ⚠️
...ntracts/src/token/erc721/extensions/uri_storage.rs 71.4% 8 Missing ⚠️
contracts/src/token/erc20/extensions/capped.rs 12.5% 7 Missing ⚠️
... and 8 more
Additional details and impacted files
Files with missing lines Coverage Δ
contracts-proc/src/interface_id.rs 94.4% <100.0%> (+0.1%) ⬆️
contracts-proc/src/lib.rs 100.0% <ø> (ø)
contracts/src/access/control.rs 98.6% <100.0%> (+1.5%) ⬆️
contracts/src/token/common/erc2981.rs 99.2% <100.0%> (+0.5%) ⬆️
contracts/src/token/erc1155/extensions/burnable.rs 100.0% <ø> (ø)
...tracts/src/token/erc1155/extensions/uri_storage.rs 100.0% <ø> (+3.0%) ⬆️
contracts/src/token/erc20/extensions/burnable.rs 100.0% <ø> (ø)
contracts/src/token/erc20/mod.rs 97.2% <100.0%> (+<0.1%) ⬆️
contracts/src/token/erc20/utils/safe_erc20.rs 53.1% <100.0%> (+2.8%) ⬆️
contracts/src/token/erc721/extensions/burnable.rs 100.0% <ø> (ø)
... and 36 more

... and 8 files with indirect coverage changes

@0xNeshi
Copy link
Collaborator

0xNeshi commented May 26, 2025 •
edited
Loading

Resolved conflicts, but couldn't push to branch due to protections, opened a new PR #671.
@bidzyyys should we close this PR then?

This was referenced May 26, 2025
Closed
Merged
@bidzyyys
Copy link
Collaborator Author

Yeah, we should close this PR as there are conflicts and we cannot merge main to v0.2.

@bidzyyys bidzyyys closed this May 26, 2025
bidzyyys added a commit that referenced this pull request May 26, 2025
@0xNeshi @bidzyyys
@dependabot @qalisander @YanVictorSN @nickventon
chore: Merge v0.2 (with conflicts resolved) into main (#671)
cc26daf 
Original PR
#669

- resolved conflicts with origin/main
- removed _check-wasm.yml_ <- if a contract is not deployable,
corresponding e2e tests would fail; we also never check and/or compare
changes in actual wasm size, we only care if it's within limits.

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Daniel Bigos <daniel.bigos@openzeppelin.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Alisander Qoshqosh <37006439+qalisander@users.noreply.github.com>
Co-authored-by: Alisander Qoshqosh <qalisander@gmail.com>
Co-authored-by: Daniel Bigos <daniel.bigos@icloud.com>
Co-authored-by: Yan Victor SN <98413246+YanVictorSN@users.noreply.github.com>
Co-authored-by: Nicholas Kelson <ambala.sweets.dev@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0xNeshi 0xNeshi 0xNeshi approved these changes

@qalisander qalisander Awaiting requested review from qalisander qalisander is a code owner

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
@bidzyyys @0xNeshi