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

Error "Failed to determine code owners for PR." when using assigneesFromCodeOwners #6432

Closed
dschafhauser opened this issue Jun 4, 2020 · 2 comments
Assignees
Labels
type:bug Bug fix of existing functionality

Comments

@dschafhauser
Copy link

dschafhauser commented Jun 4, 2020

What Renovate type are you using?
Self-hosted via docker image 'renovate/renovate:20.25.5' on own GitLab instance.

Describe the bug

I've enabled the assigneesFromCodeOwners options as described in the docs:
https://docs.renovatebot.com/configuration-options/#assigneesfromcodeowners

However, renovate consistently fails to determine code owners for the PRs it creates.

Did you see anything helpful in debug logs?

WARN: Failed to determine code owners for PR. (repository=my-repository, branch=renovate/some-dependency-)
"err": {
  "message": "Cannot read property 'every' of null",
  "stack": "TypeError: Cannot read property 'every' of null
    at /usr/src/app/node_modules/renovate/dist/workers/pr/code-owners.js:35:59
    at Array.find (<anonymous>)
    at Object.codeOwnersForPr (/usr/src/app/node_modules/renovate/dist/workers/pr/code-owners.js:35:36)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
    at async addCodeOwners (/usr/src/app/node_modules/renovate/dist/workers/pr/index.js:24:55)
    at async addAssigneesReviewers (/usr/src/app/node_modules/renovate/dist/workers/pr/index.js:29:21)
    at async Object.ensurePr (/usr/src/app/node_modules/renovate/dist/workers/pr/index.js:226:17)
    at async Object.processBranch (/usr/src/app/node_modules/renovate/dist/workers/branch/index.js:452:42)
    at async Object.writeUpdates (/usr/src/app/node_modules/renovate/dist/workers/repository/process/write.js:25:21)
    at async Object.update (/usr/src/app/node_modules/renovate/dist/workers/repository/process/extract-update.js:60:15)
    at async Object.renovateRepository (/usr/src/app/node_modules/renovate/dist/workers/repository/index.js:40:21)
    at async Object.start (/usr/src/app/node_modules/renovate/dist/workers/global/index.js:94:13)
    at async /usr/src/app/node_modules/renovate/dist/renovate.js:28:24"
}

To Reproduce

I haven't set up a minimal example to reproduce yet.

Additional context

None.

@rarkins rarkins added the type:bug Bug fix of existing functionality label Jun 4, 2020
@rarkins rarkins closed this as completed in 33ca9d9 Jun 4, 2020
@rarkins rarkins self-assigned this Jun 4, 2020
@rarkins
Copy link
Collaborator

rarkins commented Jun 4, 2020

I've fixed this particular warning, but I suspect there may be another problem, because prFiles shouldn't be empty. Cc @fgreinacher

@renovate-release
Copy link
Collaborator

🎉 This issue has been resolved in version 20.25.8 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type:bug Bug fix of existing functionality
Projects
None yet
Development

No branches or pull requests

3 participants