-
Notifications
You must be signed in to change notification settings - Fork 9
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
f: almost working #53
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,6 +5,19 @@ All notable changes to this project will be documented in this file. | |
The format is based on [Keep a Changelog](http://keepachangelog.com/) | ||
and this project adheres to [Semantic Versioning](http://semver.org/). | ||
|
||
## 1.13.3 - 2024-16-10 | ||
|
||
### Changed | ||
|
||
- Bumped `[email protected]` | ||
- Bumped `[email protected]` | ||
|
||
## 1.13.2-auction.1 - 2024-16-10 | ||
|
||
### Changed | ||
|
||
- Bumped `[email protected]` | ||
|
||
## 1.13.2-auction - 2024-08-10 | ||
|
||
### Changed | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,23 +4,23 @@ edition = "2021" | |
license = "MIT OR Apache-2.0" | ||
name = "injective-test-tube" | ||
repository = "https://github.com/InjectiveLabs/test-tube" | ||
version = "1.13.2-auction" | ||
version = "1.13.3" | ||
# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html | ||
|
||
exclude = [ "injective-core", "test_artifacts" ] | ||
|
||
[dependencies] | ||
base64 = "0.21.5" | ||
cosmrs = { version = "0.15.0", features = [ "cosmwasm", "rpc" ] } | ||
cosmrs = { version = "0.20.0", features = [ "cosmwasm", "rpc" ] } | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification Remove unnecessary prost-derive feature, versions are compatible The version updates (cosmrs, injective-std, prost) follow semantic versioning and appear compatible. The cosmrs 0.20.0 update only bumps cosmos-sdk-proto dependency. However, the added prost-derive feature appears unnecessary as no prost derive macros are used in the codebase.
🔗 Analysis chainVerify version compatibility across dependencies. Multiple major version updates:
Please ensure these versions are compatible and the new prost feature is required. Run this script to check for potential compatibility issues: Also applies to: 18-19 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Check if these versions are used together in other Injective projects
gh api graphql -f query='
{
search(query: "org:InjectiveLabs path:Cargo.toml \"cosmrs = { version = \\\"0.20.0\\\"\" \"injective-std = { version = \\\"1.13.5\\\"\"", type: CODE, first: 5) {
nodes {
... on CodeResult {
repository {
nameWithOwner
}
file {
path
}
}
}
}
}'
# Check if prost-derive feature is used in the code
rg "use prost" -A 3
Length of output: 3036 Script: #!/bin/bash
# Search for prost derive macros
rg "#\[derive\(" -A 1 | rg -B 1 "Message|Encode|Decode"
# Check Cargo.lock for resolved versions
rg "^(cosmrs|prost|injective-std)" Cargo.lock
# Try fixed GitHub query
gh search code --owner=InjectiveLabs --filename=Cargo.toml 'cosmrs = { version = "0.20.0"' --json repository,path
# Look for potential breaking changes in cosmrs
curl -s https://raw.githubusercontent.com/cosmos/cosmos-rust/main/cosmrs/CHANGELOG.md | head -n 20
Length of output: 1247 Script: #!/bin/bash
# Search specifically for prost Message derives
rg -B 1 -A 1 '(?m)^#\[derive.*Message.*\]'
# Check if prost-derive is actually used
rg 'use prost::derive|use prost_derive'
# Check if there are any direct prost type imports that might need derive
rg 'use prost::\w+::'
Length of output: 110 |
||
cosmwasm-schema = { version = "2.1.1" } | ||
cosmwasm-std = { version = "2.1.0", features = [ "abort", "cosmwasm_1_2", "cosmwasm_1_3", "cosmwasm_1_4", "cosmwasm_2_0", "iterator", "stargate" ] } | ||
hex = "0.4.2" | ||
injective-cosmwasm = { version = "0.3.0" } | ||
injective-std = { version = "=1.13.2-auction" } | ||
prost = "0.12.3" | ||
injective-std = { version = "1.13.6" } | ||
prost = { version = "0.13.4", features = [ "prost-derive" ] } | ||
serde = "1.0.144" | ||
serde_json = "1.0.85" | ||
test-tube-inj = { version = "2.0.1" } | ||
test-tube-inj = { version = "2.0.4" } | ||
thiserror = "1.0.34" | ||
|
||
[build-dependencies] | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix invalid date format.
The date format "2024-16-10" is invalid as the month value cannot be 16. Please update to a valid date in YYYY-MM-DD format.
Also applies to: 15-15