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

Manual merge required even though update_type: all #237

Open
Noxillio opened this issue Oct 9, 2023 · 1 comment
Open

Manual merge required even though update_type: all #237

Noxillio opened this issue Oct 9, 2023 · 1 comment

Comments

@Noxillio
Copy link

Noxillio commented Oct 9, 2023

The workflow is requiring manual merging despite the update_type being set to all. Nothing here points to why this is happening and I need assistance, please.

loaded merge config: 
- match:
    update_type: all

title: "Bump @sapphire/ts-config from 4.0.1 to 5.0.0"
depName: @sapphire/ts-config
from: 4.0.1
to: 5.0.0
dependency type: development
security critical: false
manual merging required

This is currently happening with all Dependabot PRs.

@Noxillio
Copy link
Author

Noxillio commented Oct 12, 2023

I figured out the problem.

Documentation indicates that dependency_type is set to all by default, and is not a required config option. This does not seem to be the case when you specify a config manually and leave the option out. The only option I was specifying, as I needed it, was update_type: all. After specifying dependency_type: all manually, the workflow began to work normally again.

Documentation may need to be updated, but this seems more like a bug to me.

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

No branches or pull requests

1 participant