No space required after WIP identifier

Fixes #2426 (closed)

Currently, the WIP: (or any other variation) prefix of a merge request title must include a trailing space for it to be recognized as a work in progress and prevent merging. To resolve this, I added an or pattern to the regexp to look for any one of the three possible delimiters ':', ']', or ' '. This way the trailing space isn't required but it still won't mark any titles beginning with wip as work in progress (e.g. "Wipwap").

Not really concerning your MR, I'm just wondering what the GL reviewer thinks of the case insensitivity here.

Added 1328 commits:

• 6e6a9906...eeba266e - 1327 commits from branch gitlab-org:master
• 9fc6df51 - Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ce into wipMergeSpacing

This has two commits now, but one is just fixing merge conflicts.

@TheTedHogan Can you retry again with CI?

@DouweM Looks good to me. Any issues with this?

I think this would be not_to, unless we add a space after WIP?

Good catch, it shouldn't be marked if it has one of the three delimiters. The case when there isn't a delimiter is handled a couple cases down, starting on 201. I've removed this case and one other redundant case

Added 1 commit:

• f0c2f748 - Removed redundent and incorrect tests on WIP

Enabled an automatic merge when the build for TheTedHogan/gitlab-ce@f0c2f7481febf1e1149cb3dba87780c462759d14 succeeds

I can't figure out why the builds are failing. It looks like they fail setting up the environment, but I'm not sure what to do to make it pass. Any ideas?

Canceled the automatic merge

Status changed to merged

mentioned in commit a1fa9731

@TheTedHogan I ran the test locally and it passed. Not sure why it's failing here, but I'll go ahead and merge.

@TheTedHogan Thanks for contributing this :)