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

Fix linting issues #641

Closed
wants to merge 1 commit into from
Closed

Conversation

humanalgorithm
Copy link
Contributor

  1. fix cargo fmt -- --check - DONE
  2. fix cargo clippy --all-targets -- -D warnings - TBD

Copy link

Cosm-Orc Gas Usage

Contract Op Name Gas Used Old Gas Used Gas Diff File
open_edition_factory Execute__factory_exec_minter_inst 547471 543794 +0.6762% e2e/src/tests/open_edition_factory_and_mint_tests.rs:206
open_edition_factory Store__Store 3506627 3489012 +0.5049% e2e/src/helpers/chain.rs:78
multiple_contracts Execute__minter_batch_exec_mint_token 3158376 18817310 -83.2156% e2e/src/tests/open_edition_factory_and_mint_tests.rs:276
base_minter Store__Store 7071995 7010297 +0.8801% e2e/src/helpers/chain.rs:78
open_edition_minter Store__Store 8701578 8627933 +0.8536% e2e/src/helpers/chain.rs:78
sg721_metadata_onchain Store__Store 8886535 8823355 +0.7161% e2e/src/helpers/chain.rs:78
sg721_nt Store__Store 7842375 7798188 +0.5666% e2e/src/helpers/chain.rs:78
vending_minter Store__Store 8896051 8840671 +0.6264% e2e/src/helpers/chain.rs:78
Raw Report for e2ec0a9
Contract Op Name Gas Used Gas Wanted File
vending_factory Instantiate__factory_inst 187383 257958 e2e/src/helpers/helper.rs:35
vending_factory Execute__factory_exec_minter_inst 34038022 51033831 e2e/src/tests/factory_test.rs:98
vending_factory Store__Store 3082476 4600574 e2e/src/helpers/chain.rs:78
vending_factory Execute__factory_exec_minter_inst_w_trading_time 3858235 5764152 e2e/src/tests/factory_test.rs:244
open_edition_factory Instantiate__factory_inst 188167 259133 e2e/src/helpers/open_edition_minter_helpers.rs:39
open_edition_factory Execute__factory_exec_minter_inst 547471 798080 e2e/src/tests/open_edition_factory_and_mint_tests.rs:206
open_edition_factory Execute__factory_exec_minter_inst_w_trading_time 548182 799146 e2e/src/tests/open_edition_factory_and_mint_tests.rs:411
open_edition_factory Store__Store 3506627 5236800 e2e/src/helpers/chain.rs:78
minter Execute__minter_exec_mint_to_token 365553 525206 e2e/src/tests/open_edition_minter_executes_tests.rs:181
minter Execute__minter_exec_purge 140240 187293 e2e/src/tests/open_edition_minter_executes_tests.rs:282
minter Execute__minter_exec_update_per_addr_limit 209877 291698 e2e/src/tests/open_edition_minter_executes_tests.rs:217
minter Execute__minter_exec_update_trading_time 271495 384122 e2e/src/tests/open_edition_minter_executes_tests.rs:100
multiple_contracts Execute__minter_batch_exec_mint_token 3158376 4714706 e2e/src/tests/open_edition_factory_and_mint_tests.rs:276
multiple_contracts Execute__minter_batch_exec_mint_token_w_trading_time 1950757 2900417 e2e/src/tests/factory_test.rs:314
base_factory Store__Store 2696792 4022048 e2e/src/helpers/chain.rs:78
base_minter Store__Store 7071995 10584792 e2e/src/helpers/chain.rs:78
open_edition_minter Store__Store 8701578 13029162 e2e/src/helpers/chain.rs:78
sg721_base Store__Store 8416507 12601560 e2e/src/helpers/chain.rs:78
sg721_metadata_onchain Store__Store 8886535 13306602 e2e/src/helpers/chain.rs:78
sg721_nt Store__Store 7842375 11740362 e2e/src/helpers/chain.rs:78
sg721_updatable Store__Store 9204723 13783884 e2e/src/helpers/chain.rs:78
sg_whitelist Store__Store 3599317 5375835 e2e/src/helpers/chain.rs:78
sg_whitelist_flex Store__Store 3528506 5269619 e2e/src/helpers/chain.rs:78
vending_minter Store__Store 8896051 13320876 e2e/src/helpers/chain.rs:78
vending_minter_merkle_wl Store__Store 9141985 13689777 e2e/src/helpers/chain.rs:78
vending_minter_wl_flex Store__Store 9078610 13594715 e2e/src/helpers/chain.rs:78
whitelist_immutable Store__Store 2206354 3286391 e2e/src/helpers/chain.rs:78
whitelist_mtree Store__Store 7049531 10551096 e2e/src/helpers/chain.rs:78

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.

2 participants