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

Add dependabot #6335

Closed
wants to merge 1 commit into from
Closed

Add dependabot #6335

wants to merge 1 commit into from

Conversation

oriontvv
Copy link

This PR adds dependabot config

@Darksonn
Copy link
Contributor

What is the motivation for this?

@Darksonn Darksonn added the A-ci Area: The continuous integration setup label Feb 10, 2024
@oriontvv
Copy link
Author

Hello. The main motivation is to remove duplicated transitive dependency crates with multiple versions for dependent crates(keep size and build time) and check from time to time in ci if any update of our dependencies could break the pipeline.

@Darksonn
Copy link
Contributor

We don't check in our Cargo.lock, so there's no need for automatic updates of Cargo.lock. As for the Cargo.toml, those should not be automatically updated. We intentionally specify the minimum versions that we are compatible with.

@Darksonn Darksonn closed this Feb 10, 2024
@oriontvv
Copy link
Author

Ok, thanks for clarification.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-ci Area: The continuous integration setup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants