Skip to content
This repository has been archived by the owner on Jul 3, 2024. It is now read-only.

231 - osmium-slither preformence issue #232

Merged
merged 10 commits into from
Feb 22, 2024

Conversation

github-actions[bot]
Copy link

Solves #231

@0xSwapFeeder 0xSwapFeeder force-pushed the bug/231-osmium-slither-preformence-issue-staging branch from 67b1d69 to 3992b5a Compare February 5, 2024 16:42
@0xSwapFeeder 0xSwapFeeder marked this pull request as ready for review February 5, 2024 16:47
@0xSwapFeeder
Copy link
Contributor

0xSwapFeeder commented Feb 5, 2024

@0xtekgrinder Please review my code

Copy link
Member

@0xtekgrinder 0xtekgrinder left a comment

Choose a reason for hiding this comment

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

Some changes, also don't forget to lint and format

remove-me-ba2314d8e5b1421b91ce.txt Outdated Show resolved Hide resolved
toolchains/solidity/core/crates/slither-server/Cargo.toml Outdated Show resolved Hide resolved
toolchains/solidity/core/crates/slither-server/src/main.rs Outdated Show resolved Hide resolved
@0xSwapFeeder
Copy link
Contributor

Well, I have ran a yarn install but it seems to not produce any changes, any clues for the CI ? @0xmemorygrinder @0xtekgrinder

Copy link
Contributor

@0xmemorygrinder 0xmemorygrinder left a comment

Choose a reason for hiding this comment

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

Minor change otherwise LGTM

toolchains/solidity/core/crates/slither-server/Cargo.toml Outdated Show resolved Hide resolved
@0xSwapFeeder 0xSwapFeeder force-pushed the bug/231-osmium-slither-preformence-issue-staging branch from 5828ac8 to a4c8a32 Compare February 22, 2024 01:01
@0xmemorygrinder 0xmemorygrinder force-pushed the bug/231-osmium-slither-preformence-issue-staging branch from edee6af to e84ce28 Compare February 22, 2024 06:07
@0xmemorygrinder 0xmemorygrinder requested review from 0xmemorygrinder and removed request for 0xtekgrinder February 22, 2024 06:26
@0xSwapFeeder 0xSwapFeeder merged commit 4df85ae into dev Feb 22, 2024
24 checks passed
@0xmemorygrinder 0xmemorygrinder deleted the bug/231-osmium-slither-preformence-issue-staging branch February 22, 2024 06:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants