Skip to content

Advisories for evm-units and uniswap-utils#2501

Merged
djc merged 4 commits intorustsec:mainfrom
carols10cents:evm-units-uniswap-utils
Feb 6, 2026
Merged

Advisories for evm-units and uniswap-utils#2501
djc merged 4 commits intorustsec:mainfrom
carols10cents:evm-units-uniswap-utils

Conversation

@carols10cents
Copy link
Contributor

My understanding is this is going to fail the linter because we've already taken down these crates from crates.io: rustsec/rustsec#1479

And I didn't add categories because of #2499 and #2500

@tarcieri
Copy link
Member

tarcieri commented Dec 5, 2025

They probably need at least a placeholder version. >= 0 should work.

@carols10cents
Copy link
Contributor Author

Looks like a bunch of crates have this:

[versions]
patched = []

@carols10cents
Copy link
Contributor Author

Ok now i think this is at the "crate doesn't exist in crates.io" lint failure?

error: error linting advisory DB .: crates.io index error: crates.io package name does not match package name in advisory for uniswap-utils in RUSTSEC-0000-0000

@tarcieri
Copy link
Member

tarcieri commented Dec 5, 2025

Yep, looks like we need rustsec/rustsec#1479 to move forward now

@djc
Copy link
Member

djc commented Dec 17, 2025

I think we're ready for this now, see

@carols10cents carols10cents force-pushed the evm-units-uniswap-utils branch from 4490aa4 to 2d72ca9 Compare February 6, 2026 15:41
@carols10cents
Copy link
Contributor Author

@djc Whoops, didn't realize yinz were waiting on me here! I've rebased and added the expect-deleted flag, so I think this is ready to merge.

@djc djc merged commit ec3e0d8 into rustsec:main Feb 6, 2026
1 check passed
@carols10cents carols10cents deleted the evm-units-uniswap-utils branch February 6, 2026 16:18
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.

3 participants