Merge pull request #724 from emacs-php/remove-link-face-in-phpdoc

Remove `'link` face in PHPDoc

Author: zonuexe

.github/workflows/test.yml

@@ -28,7 +28,7 @@ jobs:
     steps:
     - uses: actions/setup-python@v2
       with:
-        python-version: '3.6'
+        python-version: '3.11'
         architecture: 'x64'
     - uses: purcell/setup-emacs@master
       with:

CHANGELOG.md

@@ -2,7 +2,14 @@
 
 All notable changes of the PHP Mode 1.19.1 release series are documented in this file using the [Keep a CHANGELOG](https://keepachangelog.com/) principles.
 
-<!-- ## Unreleased -->
+## Unreleased
+
+### Removed
+
+ * No longer highlights `'link` in PHPDoc ([#724])
+   * Please use `goto-address-prog-mode` minor mode
+
+[#724]: https://github.com/emacs-php/php-mode/pull/724
 
 ## [1.24.2] - 2022-11-13
 

lisp/php-mode.el

@@ -1371,8 +1371,6 @@ for \\[find-tag] (which see)."
               (regexp-opt php-phpdoc-type-names 'words)
               "\\)")
      1 font-lock-type-face prepend nil)
-    ("https?://[^\n\t ]+"
-     0 'link prepend nil)
     ("^\\(?:/\\*\\)?\\(?:\\s \\|\\*\\)*\\(@[[:alpha:]][-[:alpha:]\\]*\\)" ; "@foo ..." markup.
      1 'php-doc-annotation-tag prepend nil)))
 

tests/7.4/arrow-function.php.faces

@@ -1,13 +1,9 @@
 ;; -*- mode: emacs-lisp -*-
 (("<?php" . php-php-tag)
  ("\n\n")
- ("/**\n * GitHub-Issue: " . font-lock-doc-face)
- ("https://github.com/emacs-php/php-mode/issues/506" link font-lock-doc-face)
- ("\n *\n * " . font-lock-doc-face)
+ ("/**\n * GitHub-Issue: https://github.com/emacs-php/php-mode/issues/506\n *\n * " . font-lock-doc-face)
  ("@see" php-doc-annotation-tag font-lock-doc-face)
- (" " . font-lock-doc-face)
- ("https://wiki.php.net/rfc/arrow_functions_v2" link font-lock-doc-face)
- ("\n */" . font-lock-doc-face)
+ (" https://wiki.php.net/rfc/arrow_functions_v2\n */" . font-lock-doc-face)
  ("\n\n")
  ("$" . php-variable-sigil)
  ("fn1" . php-variable-name)

tests/issue-136.php.faces

@@ -11,9 +11,7 @@
  ("@author" php-doc-annotation-tag font-lock-doc-face)
  ("    USAMI Kenta <tadsan@pixiv.com>\n * " . font-lock-doc-face)
  ("@link" php-doc-annotation-tag font-lock-doc-face)
- ("      " . font-lock-doc-face)
- ("https://github.com/emacs-php/php-mode" link font-lock-doc-face)
- ("\n * " . font-lock-doc-face)
+ ("      https://github.com/emacs-php/php-mode\n * " . font-lock-doc-face)
  ("@package" php-doc-annotation-tag font-lock-doc-face)
  ("   " . font-lock-doc-face)
  ("Emacs\\PHPMode" php-string font-lock-doc-face)

tests/issue-197.php.faces

@@ -1,9 +1,7 @@
 ;; -*- mode: emacs-lisp -*-
 (("<?php" . php-php-tag)
  ("\n\n")
- ("/**\n * GitHub Issue:    " . font-lock-doc-face)
- ("https://github.com/emacs-php/php-mode/issues/197" link font-lock-doc-face)
- ("\n *\n * Test that member highlighting goes before highlighting of cc-mode\n * types\n */" . font-lock-doc-face)
+ ("/**\n * GitHub Issue:    https://github.com/emacs-php/php-mode/issues/197\n *\n * Test that member highlighting goes before highlighting of cc-mode\n * types\n */" . font-lock-doc-face)
  ("\n")
  ("$" . php-variable-sigil)
  ("test" . php-variable-name)

tests/issue-201.php.faces

@@ -1,9 +1,7 @@
 ;; -*- mode: emacs-lisp -*-
 (("<?php" . php-php-tag)
  ("\n\n")
- ("/**\n * GitHub Issue:    " . font-lock-doc-face)
- ("https://github.com/emacs-php/php-mode/issues/201" link font-lock-doc-face)
- ("\n *\n * Test highighting of " . font-lock-doc-face)
+ ("/**\n * GitHub Issue:    https://github.com/emacs-php/php-mode/issues/201\n *\n * Test highighting of " . font-lock-doc-face)
  ("$" php-doc-$this-sigil php-doc-variable-sigil font-lock-doc-face)
  ("this" php-doc-$this php-variable-name font-lock-doc-face)
  ("\n */" . font-lock-doc-face)

tests/issue-305.php.faces

@@ -1,9 +1,7 @@
 ;; -*- mode: emacs-lisp -*-
 (("<?php" . php-php-tag)
  ("\n\n")
- ("/**\n * GitHub Issue:    " . font-lock-doc-face)
- ("https://github.com/emacs-php/php-mode/issues/305" link font-lock-doc-face)
- ("\n *\n * Test highighting of " . font-lock-doc-face)
+ ("/**\n * GitHub Issue:    https://github.com/emacs-php/php-mode/issues/305\n *\n * Test highighting of " . font-lock-doc-face)
  ("$" php-doc-variable-sigil font-lock-doc-face)
  ("this_foo" php-variable-name font-lock-doc-face)
  (", " . font-lock-doc-face)

tests/issue-439.php.faces

@@ -1,9 +1,7 @@
 ;; -*- mode: emacs-lisp -*-
 (("<?php" . php-php-tag)
  ("\n\n")
- ("/**\n * GitHub Issue:  " . font-lock-doc-face)
- ("https://github.com/emacs-php/php-mode/issues/439" link font-lock-doc-face)
- ("\n */" . font-lock-doc-face)
+ ("/**\n * GitHub Issue:  https://github.com/emacs-php/php-mode/issues/439\n */" . font-lock-doc-face)
  ("\n\n")
  ("$" . php-variable-sigil)
  ("a" . php-variable-name)

tests/issue-443.php.27.faces

@@ -8,9 +8,7 @@
  ("1); ")
  ("?>" . php-php-tag)
  ("\n")
- ("/** GitHub Issue: " . font-lock-doc-face)
- ("https://github.com/emacs-php/php-mode/issues/443" link font-lock-doc-face)
- (" */" . font-lock-doc-face)
+ ("/** GitHub Issue: https://github.com/emacs-php/php-mode/issues/443 */" . font-lock-doc-face)
  ("\n")
  ("<?xml" . php-php-tag)
  (" version")