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

Turn on dependabot to roll bundler dependencies #71602

Merged
merged 3 commits into from Dec 3, 2020

Conversation

jmagman
Copy link
Member

@jmagman jmagman commented Dec 2, 2020

Description

Turn on GitHub dependabot to keep the dev/ci Gemfile.lock files up to date. Run it weekly.

Related Issues

See #71531 where this was done manually.
It's now opt-in, but for a brief window we were getting dependabot PRs, see #62379

Tests

Might not know if this works until next Monday.

@jmagman jmagman added team Infra upgrades, team productivity, code health, technical debt. See also team: labels. team-infra Owned by Infrastructure team labels Dec 2, 2020
@jmagman jmagman self-assigned this Dec 2, 2020
@google-cla google-cla bot added the cla: yes label Dec 2, 2020
schedule:
interval: "weekly"
reviewers:
- "jmagman"
Copy link
Member Author

Choose a reason for hiding this comment

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

@godofredoc Should someone from infra be added as a backup?

Copy link
Contributor

Choose a reason for hiding this comment

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

sounds good, please add @keyonghan and @yusufm

labels:
- "team"
- "team: infra"
- "waiting for tree to go green"
Copy link
Member Author

@jmagman jmagman Dec 2, 2020

Choose a reason for hiding this comment

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

Will add dependabot to the autorollers list so we can mostly ignore it if the tests pass.

labels:
- "team"
- "team: infra"
- "waiting for tree to go green"
Copy link
Member

Choose a reason for hiding this comment

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

;)

Copy link
Member

@christopherfujino christopherfujino left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team Infra upgrades, team productivity, code health, technical debt. See also team: labels. team-infra Owned by Infrastructure team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants