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(clippy): fix coins mod clippy warnings in wasm #2224

Merged
merged 2 commits into from
Sep 23, 2024

Conversation

borngraced
Copy link
Member

No description provided.

@borngraced borngraced self-assigned this Sep 16, 2024
@onur-ozkan
Copy link
Member

Looking at dev CI, there are no linting errors at all ? 🤔

@Alrighttt
Copy link
Member

Looking at dev CI, there are no linting errors at all ? 🤔

It's a fix for --all-features case.

@onur-ozkan
Copy link
Member

Looking at dev CI, there are no linting errors at all ? 🤔

It's a fix for --all-features case.

We should include this flag to CI then.

@borngraced
Copy link
Member Author

Looking at dev CI, there are no linting errors at all ? 🤔

It's a fix for --all-features case.

We should include this flag to CI then.

those clippy warnings can also show up when running

cargo test --target wasm32-unknown-unknown --package coins

--all-features is unusable now until SiaCode http_client code is synced

@shamardy shamardy mentioned this pull request Sep 20, 2024
5 tasks
@shamardy
Copy link
Collaborator

--all-features is unusable now until SiaCode http_client code is synced

Added it to the SIA tracking issue #2201 (comment) c.c. @borngraced @Alrighttt

@shamardy shamardy changed the title fix(chore): coins mod WASM clippy warnings fix(clippy): coins mod WASM clippy warnings Sep 20, 2024
@shamardy shamardy changed the title fix(clippy): coins mod WASM clippy warnings fix(clippy): fix coins mod WASM clippy warnings Sep 20, 2024
@shamardy shamardy changed the title fix(clippy): fix coins mod WASM clippy warnings fix(clippy): fix coins mod clippy warnings in wasm Sep 20, 2024
…to the only place it's used

* use not wasm attribute for solana macros
Copy link
Collaborator

@shamardy shamardy left a comment

Choose a reason for hiding this comment

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

Will merge once @onur-ozkan accepts this.

@shamardy shamardy merged commit cde7f36 into dev Sep 23, 2024
23 of 26 checks passed
@shamardy shamardy deleted the fix-coins-mod-wasm-clippy branch September 23, 2024 10:28
@mariocynicys
Copy link
Collaborator

those clippy warnings can also show up when running
cargo test --target wasm32-unknown-unknown --package coins
--all-features is unusable now until SiaCode http_client code is synced

I think we want to run these tests in CI. Looks like they are never ran.

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.

5 participants