-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Automerge patch not working? #720
Comments
I set up a test here: renovate-tests/automerge#10 using your renovate.json config and travis-ci It behaved as following:
@denkristoffer I notice in your repo that auth0-js is now up to 8.9.3. Did you merge that manually, or did renovate automerge it? In your screenshot it shows 13 hours elapsed since the PR creation and still no automerge, so it looks like something was wrong. There's a reasonable chance that whatever was wrong was fixed in this commit this morning: 2ca2e7c |
I merged it manually. I'll keep an eye on whether the commit this morning changed anything from here on. |
Ok, please reopen this issue if you find it happening again |
It's happening again 🙂
I've left it unmerged. Oh, is it possible that it's due to warnings? Since the workspace warning is always in the PR. |
I think it's a config problem. I'm seeing:
I also found these log messages:
It looks like you have 2 package.json files but the config is in only one of them. Therefore any config you specify will be applied only to packages in that package.json and not to the other one. In theory we might be able to add some "overrideRepoConfig: true" option in future that lets one package.json specify the behaviour for all package files, but that's not supported today. You will need to either copy/paste the config into the second package.json, or into a renovate.json file in the root of the repo. |
Oh indeed, the renovate config is only in my root package.json, I just assumed that it would apply to the whole repository when placed there. I'll add the config to the different packages and see if that fixes it 👍 |
Transferred from #473 (comment):
Another thing:
"automerge": "minor"
– shouldn't this automerge commits such as this one, instead of creating a PR?:The text was updated successfully, but these errors were encountered: