8000 build: Reconfigure dependabot by theofidry · Pull Request #1376 · phpbrew/phpbrew · GitHub
[go: up one dir, main page]

Skip to content

build: Reconfigure dependabot#1376

Open
theofidry wants to merge 1 commit intophpbrew:masterfrom
theofidry:build/dependabots
Open

build: Reconfigure dependabot#1376
theofidry wants to merge 1 commit intophpbrew:masterfrom
theofidry:build/dependabots

Conversation

@theofidry
Copy link
Contributor
  • Configure it to be executed weekly only
  • Check GitHub Actions too

@peter279k peter279k requested a review from c9s June 24, 2024 02:20
labels:
- Dependencies
versioning-strategy: increase
- package-ecosystem: "composer"
Copy link
Member
@peter279k peter279k Jun 24, 2024

Choose a reason for hiding this comment

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

Some settings are removed. Could you add them back?Or explain why they should be removed.

Such as open-pull-requests-limit, reviewers , assignees and labels.

Copy link
Contributor Author
@theofidry theofidry Jun 24, 2024

Choose a reason for hiding this comment

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

open-pull-requests-limit

I don't think it's necessary: there won't be that many and in my experience, with a similar config, dependabot will close and re-open MRs so there will not be many in parallel.

Do note that a difference here is that it does one big PR with upgrades. In my experience it can get pretty spammy so having a default big PR that pops up once a week at most is easier to manage IMO. If there was a problematic dependency to update, at least for me, I just took care of it manually separately and asked dependabot to rebase its PR afterwards.

So it's not the most rigorous, but I think it's a pretty good compromise especially on projects with lower bandwith availability.

reviewers, assignees

I don't think it's up to date, and is there really a point? However has access can check it out anytime instead.

labels

The PRs already get build(deps): as a name prefix, if you still want the label we can add it, I just saw no value in it.

@theofidry theofidry mentioned this pull request Jun 24, 2024
2 tasks
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

Comments

0