Skip to content

[CI] Add Danger review to perform automated MR checks

Description

This MR introduces Danger as a job to our ci. This allows us to perform several automated checks on the MR itself. Essentially, it is aimed to be a replacement for any comments of the type "don't forget to ....".

  • Title
    • Must begin with one of the allowed scope prefixes: [CI], [Misc], [Tools], [catalogue], [frontend], [scheduler], [taped], [rmcd]
    • Must be 72 characters or fewer (soft limit)
  • Description
    • Must not be empty
    • Must include a properly filled-out ### Description section
    • Must not contain the placeholder comment (...Mandatory: provide a description...)
    • Should be at least 40 characters (soft check)
    • Must include all required checklist acknowledgements:
      • - [x] Documentation reflects the changes made.
      • - [x] Merge Request title is clear, concise, and suitable as a changelog entry.
  • Labels
    • Must include exactly one type:: label (e.g., type::bug)
    • Must include exactly one workflow:: label (e.g., workflow::testing)
    • Must include exactly one priority:: label (e.g., priority::high)
    • Should include at least one additional label for scoping/context
  • Assignee
    • Must have an assignee set
  • Reviewers
    • Should have at least one reviewer assigned
  • Merge Request Size
    • Warns if more than 500 lines are changed

Checklist

  • Documentation reflects the changes made.
  • Merge Request title is clear, concise, and suitable as a changelog entry. See this link

References

Closes #1195 (closed)

Edited by Niels Alexander Buegel

Merge request reports

Loading