does this fix lgtm?

alanorozco · alanorozco · commit 721ef27e68f9 · 2021-11-04T13:47:56.000-07:00
diff --git a/extensions/amp-story/1.0/amp-story-page-attachment.js b/extensions/amp-story/1.0/amp-story-page-attachment.js
@@ -219,11 +219,12 @@ export class AmpStoryPageAttachment extends DraggableDrawer {
       );
 
     // Use an anchor element to make this a real link in vertical rendering.
-    // TODO(wg-stories): Evaluate whether this link needs noopener or noreferrer.
-    // See https://lgtm.com/rules/1806963085/ and remove disabling comment for js/unsafe-external-link
     const link = (
+      // TODO(wg-stories): Evaluate whether this link needs noopener or noreferrer.
+      // See https://lgtm.com/rules/1806963085/ and remove disabling comment below.
+      // lgtm [js/unsafe-external-link]
       <a
-        class="i-amphtml-story-page-attachment-remote-content" // lgtm [js/unsafe-external-link]
+        class="i-amphtml-story-page-attachment-remote-content"
         target="_blank"
         // URL will be validated and resolved based on the canonical URL if
         // relative when navigating.
