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

chore(deps): update dependency phpstan/phpstan to v1.10.56 #1513

Merged
merged 1 commit into from
Jan 15, 2024

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Jan 15, 2024

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
phpstan/phpstan 1.10.55 -> 1.10.56 age adoption passing confidence

Release Notes

phpstan/phpstan (phpstan/phpstan)

v1.10.56

Compare Source

Major new feature 🚀

Improvements 🔧

Bugfixes 🐛

Function signature fixes 🤖

Internals 🔍


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Enabled.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@spawnia
Copy link
Collaborator

spawnia commented Jan 15, 2024

image

@simPod Why does it not automerge though?

@renovate renovate bot merged commit e1f8040 into master Jan 15, 2024
13 checks passed
@renovate renovate bot deleted the renovate/phpstan branch January 15, 2024 13:22
@simPod
Copy link
Collaborator

simPod commented Jan 15, 2024

It created the PR but it could not merge since there were CI jobs pending. I tried to manually trigger next renovate run and it auto merged.

@spawnia
Copy link
Collaborator

spawnia commented Jan 15, 2024

When I looked at the PR, it looked like all CI jobs had passed successfully though? What am I missing?

@simPod
Copy link
Collaborator

simPod commented Jan 15, 2024

Yup but not at the moment of Renovate run. Renovate creates PR and within short moments checks for merge eligibility in a single run. So it needed another run after CI passed.

@spawnia
Copy link
Collaborator

spawnia commented Jan 16, 2024

So it will probably never automerge? Can we improve this somehow?

@simPod
Copy link
Collaborator

simPod commented Jan 16, 2024

It will. You can see the run log here https://developer.mend.io/github/webonyx/graphql-php

When the PR is created during the run and then CI passes, it will be auto-merged on second run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants