Skip to content

Escape usernames in triage report

Lukas Eipert requested to merge leipert-escape-user-names-in-template into master

What does this MR do and why?

With the new "multiple todo" feature flag enabled, the assignment and mention in triages leads to the creation of multiple todos for the triage report. In order to prevent this, we escape the usernames after the Hi! in order to still keep it personal, but just trigger one ping via the assignment event.

Also: add tests for the functions touched / introduced to make undercoverage happy.

Changelog: changed

Expected impact & dry-runs

These are strongly recommended to assist reviewers and reduce the time to merge your change.

Action items

  • (If applicable) Add documentation to the handbook pages for Triage Operations =>
  • (If applicable) Identify the affected groups and how to communicate to them:
    • /cc @person_or_group =>
    • Relevant Slack channels =>
    • Engineering week-in-review
Edited by Lukas Eipert

Merge request reports