search

danger / kotlin

⚠️ Stop saying "you forgot to …" in code review
MIT License
505 stars 50 forks source link

Make user nullable #235

Closed vchernyshov closed 2 years ago

vchernyshov commented 2 years ago

In some cases PR could contain commits from non BB user, in this case api could not recognise such user and as the result user property is null, as model has non nullable property Danger Kotlin simple crashes. This PR fixes this corner case.

vchernyshov commented 2 years ago

No problem) It would be great to get a fixed version asap, we have to disable Danger Kotlin for our pipeline because of this issue