Skip to content

Check cargo dependencies#2501

Merged
pkwarren merged 1 commit into
mainfrom
pkw/check-cargo
Jun 1, 2026
Merged

Check cargo dependencies#2501
pkwarren merged 1 commit into
mainfrom
pkw/check-cargo

Conversation

@pkwarren
Copy link
Copy Markdown
Member

@pkwarren pkwarren commented Jun 1, 2026

Similar to Maven, ensure that all cargo dependencies point to valid versions.

Similar to Maven, ensure that all cargo dependencies point to valid
versions.
@pkwarren pkwarren requested a review from oliversun9 June 1, 2026 15:02
Comment thread tests/plugins_test.go
break
}
}
assert.Truef(t, matched, "no non-yanked version of crate %q satisfies req %q", dep.Name, dep.VersionRequirement)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This would've caught the previous upgrade issue:

--- FAIL: TestCargoDependencies (1.05s)
    --- FAIL: TestCargoDependencies/anthropics/connect-rust@v0.6.1 (0.86s)
        plugins_test.go:478:
                Error Trace:    /Users/pkw/dev/plugins/tests/plugins_test.go:478
                Error:          Should be true
                Test:           TestCargoDependencies/anthropics/connect-rust@v0.6.1
                Messages:       no non-yanked version of crate "buffa" satisfies req "0.6.1"
        plugins_test.go:478:
                Error Trace:    /Users/pkw/dev/plugins/tests/plugins_test.go:478
                Error:          Should be true
                Test:           TestCargoDependencies/anthropics/connect-rust@v0.6.1
                Messages:       no non-yanked version of crate "buffa-types" satisfies req "0.6.1"

@pkwarren pkwarren merged commit 3369713 into main Jun 1, 2026
4 checks passed
@pkwarren pkwarren deleted the pkw/check-cargo branch June 1, 2026 15:08
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