-
Notifications
You must be signed in to change notification settings - Fork 1
Update all the dependencies #422
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
Conversation
|
I think this is a little misleading: If I understand correctly, it's saying the old config is invalid, but the new config is valid (see "Configuration changed — The new Mergify configuration is valid" in the successful checks). So, despite the fact that this PR has an unsuccessful check, it should still be ok to merge if everyone is ok with the actual changes. |
|
sounds good, looks like this will update a ton of dependencies so I'm okay merging it. |

This pulls in the commits from all the open PRs and makes the necessary changes to get everything working with the updated dependencies. It also includes better tracing and a test app that allows users to simulate running the handler locally.