danger / kotlin

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

Fix using `head_pipeline` instead of `pipeline` in GitLab #289

Closed grine4ka closed 1 month ago

grine4ka commented 1 month ago

Due to GitLab docs we should prefer using head_pipeline instead of pipeline as it contains more information.

Additional to that, sometimes I caught crashes on danger-kotlin when pipeline status was incorrect in that field or even when pipeline field was null. But I can't just mark pipeline field as nullable, cause it will cause backward incompatibility.