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

polkadot-omni-node-lib: remove unused dep #6889

Merged
merged 6 commits into from
Dec 16, 2024

Conversation

iulianbarbu
Copy link
Contributor

Description

Redundant dep that made its way in #6450 . 😅. It can be brought up when using cargo udeps. Added a github action that runs cargo udeps on the repo too.

Integration

N/A

Review Notes

N/A

Signed-off-by: Iulian Barbu <[email protected]>
@iulianbarbu iulianbarbu requested review from skunert and lexnv December 13, 2024 17:40
@iulianbarbu iulianbarbu self-assigned this Dec 13, 2024
@iulianbarbu iulianbarbu added the R0-silent Changes should not be mentioned in any release notes label Dec 13, 2024
@iulianbarbu
Copy link
Contributor Author

/cmd prdoc --bump patch --audience node_dev

@iulianbarbu iulianbarbu added T9-cumulus This PR/Issue is related to cumulus. and removed R0-silent Changes should not be mentioned in any release notes labels Dec 13, 2024
Signed-off-by: Iulian Barbu <[email protected]>
@iulianbarbu iulianbarbu changed the title remove redundant dep polkadot-omni-node-lib: remove unused dep Dec 13, 2024
@iulianbarbu iulianbarbu requested a review from a team December 13, 2024 18:10
Signed-off-by: Iulian Barbu <[email protected]>
prdoc/pr_6889.prdoc Outdated Show resolved Hide resolved
@bkchr bkchr enabled auto-merge December 16, 2024 09:03
@bkchr bkchr added this pull request to the merge queue Dec 16, 2024
Merged via the queue into paritytech:master with commit adc0178 Dec 16, 2024
197 of 200 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T9-cumulus This PR/Issue is related to cumulus.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants