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

[WIP] Add features flag for cargo stylus check/deploy #32

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cygaar
Copy link

@cygaar cygaar commented Apr 15, 2024

Description

It would be convenient if developers could specify which features they want to include when checking/deploying their stylus contracts. In my case, I'm working on a set of common contracts (similar to Openzeppelin, Solmate, Solady etc) in Rust where I gate different libraries (ERC20, ERC721, ERC1155, etc) through crate features.

The core issue is that I have multiple example contracts each with an entrypoint. By adding a features flag to cargo stylus, I'll be able to check different features easily without having to manually build and specify a wasm-path when running the check command.

Checklist

  • I have documented these changes where necessary.
  • I have read the DCO and ensured that these changes comply.
  • I assign this work under its open source licensing.

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.

1 participant