Skip to content

Lint the MR title without WIP

Dan Davison requested to merge danger-dont-look-at-wip into master

What does this MR do?

Found in !25539 ,

It turns out that the linter was checking the full MR title including WIP: , resulting in a warning

The merge request title length is acceptable, but please try to reduce it to 50 characters.

"WIP: Consider a Quality reviewer for frontend changes".size #=> 53
"Consider a Quality reviewer for frontend changes".size #=> 48

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

I'm not seeing specs to modify for this particular commit_message Dangerfile

Edited by Dan Davison

Merge request reports