Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

.github: Add dependabot config and thus enable dependabot #1937

Merged
2 commits merged into from
Nov 17, 2020

Conversation

mxinden
Copy link
Contributor

@mxinden mxinden commented Nov 9, 2020

Just like paritytech/substrate#7509 I am suggesting to enable Dependabot for the Polkadot repository.

With this pull request I suggest to enable Dependabot for this repository. Dependabot will scan the dependencies used in Substrate (both Cargo.toml and Cargo.lock) and create pull requests whenever newer versions of those dependencies are available.

We are using Dependabot in rust-libp2p since about 2 months now. While annoying at first, I think it is well worth it, reducing the long-term toil keeping dependencies up-to-date as well as staying in sync with recent (security) bug fixes.

Please don't just merge here but signal your consent / dissent for now. Keeping dependencies up-to-date is a lot of work and thus a team effort.

@mxinden mxinden added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Nov 9, 2020
@ordian
Copy link
Member

ordian commented Nov 9, 2020

I think this is a good idea in general to keep the dependencies up to date. It also helps to identify problems with a dependency update early on.

@bkchr
Copy link
Member

bkchr commented Nov 9, 2020

Same comment as on substrate.

.github/dependabot.yml Show resolved Hide resolved
Co-authored-by: Pierre Krieger <pierre.krieger1708@gmail.com>
@mxinden
Copy link
Contributor Author

mxinden commented Nov 17, 2020

Large dependency update for Polkadot happened in #1946.

@mxinden
Copy link
Contributor Author

mxinden commented Nov 17, 2020

bot merge

@ghost
Copy link

ghost commented Nov 17, 2020

Trying merge.

@ghost ghost merged commit 4cb5025 into master Nov 17, 2020
@ghost ghost deleted the mxinden-dependabot branch November 17, 2020 10:20
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants