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
fix(manager/gomod): resolve multi-line indirect dependencies #19442
Conversation
a17dbcd
to
6ff7b71
Compare
Coverage is failing due to: From this line of code which has changed because we're now not triggering this when we've got an Thoughts on what's best to do here? Trying to think of what valid |
As noticed in renovatebot#19440, the handling for indirect dependencies didn't quite work when using a multi-line `require` block. As we're now no longer hitting the `No multi-line match` logging statement when we're receiving an `indirect` dependency, we need to add a test to exercise that for our coverage requirements, too. Closes renovatebot#19440.
Head branch was pushed to by a user without write access
ef47203
to
c85c28e
Compare
Would an empty line or a line with a comment work here? I guess it's ok to remove that |
I've managed to trigger it with an empty line, yep - happy to keep it as that, or remove it if you'd prefer? |
Keep as is, who knows if it can be useful for tracing in future |
🎉 This PR is included in version 34.59.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Changes
As noticed in #19440, the handling for indirect dependencies didn't
quite work when using a multi-line
require
block.Closes #19440.
Context
dryRun: extract
#19440Documentation (please check one with an [x])
How I've tested my work (please select one)
I have verified these changes via: