08:21:56  stdout: 
08:21:56  --- /pagure/pagure/lib/notify.py  2020-09-04 07:57:46.040486 +0000
08:21:56  +++ /pagure/pagure/lib/notify.py  2020-09-04 08:21:17.155764 +0000
08:21:56  @@ -233,12 +233,13 @@
08:21:56   
08:21:56   def _add_mentioned_users(emails, comment):
08:21:56       """ Check the comment to see if an user is mentioned in it and if
08:21:56       so add this user to the list of people to notify.
08:21:56       """
08:21:56  -    filtered_comment = re.sub(FencedBlockPreprocessor.FENCED_BLOCK_RE,
08:21:56  -                              "", comment)
08:21:56  +    filtered_comment = re.sub(
08:21:56  +        FencedBlockPreprocessor.FENCED_BLOCK_RE, "", comment
08:21:56  +    )
08:21:56       for username in re.findall(MENTION_RE, filtered_comment):
08:21:56           user = pagure.lib.query.search_user(flask.g.session, username=username)
08:21:56           if user:
08:21:56               emails.add(user.default_email)
08:21:56       return emails
08:21:56  --- /pagure/tests/test_pagure_lib_notify.py   2020-09-04 07:57:46.040486 +0000
08:21:56  +++ /pagure/tests/test_pagure_lib_notify.py   2020-09-04 08:21:39.417395 +0000
08:21:56  @@ -546,13 +546,16 @@
08:21:56   
08:21:56       def test_notification_mention(self):
08:21:56           g = munch.Munch()
08:21:56           g.session = self.session
08:21:56           with patch("flask.g", g):
08:21:56  +
08:21:56               def _check_mention(comment, exp):
08:21:56                   emails = set([])
08:21:56  -                emails = pagure.lib.notify._add_mentioned_users(emails, comment)
08:21:56  +                emails = pagure.lib.notify._add_mentioned_users(
08:21:56  +                    emails, comment
08:21:56  +                )
08:21:56   
08:21:56                   self.assertEqual(emails, exp)
08:21:56   
08:21:56               exp = set(["bar@pingou.com"])
08:21:56               comment = "I think we should ask @pingou how to pronounce pagure"
08:21:56  
08:21:56  stderr: 
08:21:56  would reformat /pagure/pagure/lib/notify.py
08:21:56  would reformat /pagure/tests/test_pagure_lib_notify.py
08:21:56  Oh no! 💥 💔 💥
08:21:56  2 files would be reformatted, 197 files would be left unchanged.

````
foo bar
````

diff --git a/ tests/test_pagure_lib_notify_email.py b/ tests/test_pagure_lib_notify_email.py
index b0d9ea4e9..da211ebfd 100644
--- a/tests/test_pagure_lib_notify_email.py    
+++ b/tests/test_pagure_lib_notify_email.py    
@@ -255,6 +255,106 @@ http://localhost.localdomain/test/issue/1
         # Mail should be from user1 (who wrote the comment).
         self.assertEqual(kwargs["user_from"], self.user1.fullname)
+    @mock.patch("pagure.lib.notify.send_email")
+    def test_user_notified_new_comment_not_in_code_block(self, fakemail):
+        """Ensures that @mention doesn't over-reach in code-blocks."""
+        self.comment1.comment = "So apparently they said ``@foo.com is awesome`` :)"
+        g = munch.Munch()
+        g.fas_user = tests.FakeUser(username="pingou")
+        g.authenticated = True
+        g.session = self.session
+        with mock.patch("flask.g", g):
+            pagure.lib.notify.notify_new_comment(self.comment1)
+
+        (_, args, kwargs) = fakemail.mock_calls[0]
+
+        # Mail should be sent to both users
+        self.assertEqual(args[2], "bar@pingou.com")
+
+        # Mail ID should be comment #1's mail ID...
+        self.assertEqual(kwargs["mail_id"], self.comment1.mail_id)
+
+        # In reply to issue #1's mail ID.
+        self.assertEqual(kwargs["in_reply_to"], self.issue1.mail_id)
+
+        # Project name should be...project (full) name.
+        self.assertEqual(kwargs["project_name"], self.project1.fullname)
+
+        # Mail should be from user1 (who wrote the comment).
+        self.assertEqual(kwargs["user_from"], self.user1.fullname)
+
+    @mock.patch("pagure.lib.notify.send_email")
+    def test_user_notified_new_comment_not_in_code_block2(self, fakemail):
+        """Ensures that @mention doesn't over-reach in code-blocks."""
+        self.comment1.comment = """
+So apparently they said
+````
+@foo.com is awesome and @foo is great
+
+We all love @foo !
+```
+:)
+"""
+        g = munch.Munch()
+        g.fas_user = tests.FakeUser(username="pingou")
+        g.authenticated = True
+        g.session = self.session
+        with mock.patch("flask.g", g):
+            pagure.lib.notify.notify_new_comment(self.comment1)
+
+        (_, args, kwargs) = fakemail.mock_calls[0]
+
+        # Mail should be sent to both users
+        self.assertEqual(args[2], "bar@pingou.com")
+
+        # Mail ID should be comment #1's mail ID...
+        self.assertEqual(kwargs["mail_id"], self.comment1.mail_id)
+
+        # In reply to issue #1's mail ID.
+        self.assertEqual(kwargs["in_reply_to"], self.issue1.mail_id)
+
+        # Project name should be...project (full) name.
+        self.assertEqual(kwargs["project_name"], self.project1.fullname)
+
+        # Mail should be from user1 (who wrote the comment).
+        self.assertEqual(kwargs["user_from"], self.user1.fullname)
+
+    @mock.patch("pagure.lib.notify.send_email")
+    def test_user_notified_new_comment_not_in_code_block3(self, fakemail):
+        """Ensures that @mention doesn't over-reach in code-blocks."""
+        self.comment1.comment = """
+So apparently they said
+~~~~
+@foo.com is awesome and @foo is great
+
+We all love @foo !
+~~~~
+:)
+"""
+        g = munch.Munch()
+        g.fas_user = tests.FakeUser(username="pingou")
+        g.authenticated = True
+        g.session = self.session
+        with mock.patch("flask.g", g):
+            pagure.lib.notify.notify_new_comment(self.comment1)
+
+        (_, args, kwargs) = fakemail.mock_calls[0]
+
+        # Mail should be sent to both users
+        self.assertEqual(args[2], "bar@pingou.com")
+
+        # Mail ID should be comment #1's mail ID...
+        self.assertEqual(kwargs["mail_id"], self.comment1.mail_id)
+
+        # In reply to issue #1's mail ID.
+        self.assertEqual(kwargs["in_reply_to"], self.issue1.mail_id)
+
+        # Project name should be...project (full) name.
+        self.assertEqual(kwargs["project_name"], self.project1.fullname)
+
+        # Mail should be from user1 (who wrote the comment).
+        self.assertEqual(kwargs["user_from"], self.user1.fullname)
+
     @mock.patch("pagure.lib.notify.send_email")
     def test_notify_new_issue_namespaced(
         self, fakemail