From 27cdf3e9959fbbd8ee81c6b2abcd2af7b74d71bc Mon Sep 17 00:00:00 2001
From: David Roe <roed.math@gmail.com>
Date: Wed, 8 Feb 2023 01:58:42 +0100
Subject: [PATCH] Remove comment numbers

---
 .github/workflows/ci-wsl.yml                                 | 2 +-
 build/pkgs/ecl/SPKG.rst                                      | 2 +-
 build/pkgs/ecm/spkg-install.in                               | 4 ++--
 build/pkgs/fflas_ffpack/spkg-install.in                      | 2 +-
 build/pkgs/gap/spkg-install.in                               | 2 +-
 build/pkgs/gdb/distros/conda.txt                             | 2 +-
 build/pkgs/glpk/patches/error_recovery.patch                 | 2 +-
 build/pkgs/glpk/spkg-install.in                              | 4 ++--
 build/pkgs/linbox/spkg-install.in                            | 2 +-
 build/pkgs/maxima/spkg-install.in                            | 2 +-
 build/pkgs/meataxe/spkg-install.in                           | 2 +-
 build/pkgs/numpy/spkg-install.in                             | 2 +-
 build/pkgs/ptyprocess/install-requires.txt                   | 2 +-
 build/sage_bootstrap/uncompress/tar_file.py                  | 2 +-
 src/sage/arith/constants.pxd                                 | 2 +-
 src/sage/arith/long.pxd                                      | 2 +-
 src/sage/combinat/finite_state_machine.py                    | 2 +-
 src/sage/cpython/atexit.pyx                                  | 2 +-
 src/sage/groups/additive_abelian/additive_abelian_wrapper.py | 2 +-
 src/sage/matrix/matrix_polynomial_dense.pyx                  | 2 +-
 src/sage/misc/compat.py                                      | 2 +-
 src/sage/schemes/elliptic_curves/ell_field.py                | 2 +-
 src/sage/schemes/elliptic_curves/ell_torsion.py              | 2 +-
 src/sage_docbuild/utils.py                                   | 4 ++--
 24 files changed, 27 insertions(+), 27 deletions(-)

diff --git a/.github/workflows/ci-wsl.yml b/.github/workflows/ci-wsl.yml
index e6d4d30cfc3..782397bb7be 100644
--- a/.github/workflows/ci-wsl.yml
+++ b/.github/workflows/ci-wsl.yml
@@ -13,7 +13,7 @@ jobs:
   windows:
     runs-on: windows-latest
     name: Ubuntu 20.04
-    # Following https://github.com/sagemath/sage/issues/25206#comment:63
+    # Following https://github.com/sagemath/sage/issues/25206
     steps:
     - name: Configure git
       run: git config --global core.symlinks true
diff --git a/build/pkgs/ecl/SPKG.rst b/build/pkgs/ecl/SPKG.rst
index 5752a7bd7f5..88599f93559 100644
--- a/build/pkgs/ecl/SPKG.rst
+++ b/build/pkgs/ecl/SPKG.rst
@@ -43,7 +43,7 @@ Special Update/Build Instructions
 
 -  Do NOT quote SAGE_LOCAL when setting CPPFLAGS and/or LDFLAGS,
    in spkg-install as this caused the build to break. See
-   https://github.com/sagemath/sage/issues/10187#comment:117
+   https://github.com/sagemath/sage/issues/10187
 
 -  TODO: Add the ECL test suite, and an spkg-check file to run it.
 -  TODO: Make ECL use Sage's Boehm GC on MacOS X as well (but perhaps
diff --git a/build/pkgs/ecm/spkg-install.in b/build/pkgs/ecm/spkg-install.in
index b60230f368a..3b178e04e11 100644
--- a/build/pkgs/ecm/spkg-install.in
+++ b/build/pkgs/ecm/spkg-install.in
@@ -92,10 +92,10 @@ else
             # Don't add debug symbols because configure otherwise
             # fails due to a bus error in Apple's 'ld' when trying
             # to determine if global symbols are prefixed with an
-            # underscore (cf. https://github.com/sagemath/sage/issues/5847#comment:35 ff.):
+            # underscore (cf. https://github.com/sagemath/sage/issues/5847 ff.):
             echo >&2 "Warning: Disabling debug symbols on MacOS X 10.5" \
                 "PowerPC because of a linker (?) bug."
-            echo >&2 "See https://github.com/sagemath/sage/issues/5847#comment:35" \
+            echo >&2 "See https://github.com/sagemath/sage/issues/5847" \
                 "ff. for details."
             echo >&2
             CFLAGS="-O3 $ORIGINAL_CFLAGS"
diff --git a/build/pkgs/fflas_ffpack/spkg-install.in b/build/pkgs/fflas_ffpack/spkg-install.in
index b9dd851875f..2c292fff1b1 100644
--- a/build/pkgs/fflas_ffpack/spkg-install.in
+++ b/build/pkgs/fflas_ffpack/spkg-install.in
@@ -30,7 +30,7 @@ if [ -z "$CONFIG_SHELL" ]; then
 fi
 
 # We disable openmp because of build failures, see
-# https://github.com/sagemath/sage/issues/17635#comment:67
+# https://github.com/sagemath/sage/issues/17635
 sdh_configure --with-default="$SAGE_LOCAL" --with-blas-libs="$LINBOX_BLAS" \
               "$LINBOX_BLAS_CFLAGS" --disable-static \
               --enable-precompilation $FFLAS_FFPACK_CONFIGURE
diff --git a/build/pkgs/gap/spkg-install.in b/build/pkgs/gap/spkg-install.in
index 30320027274..e11c5823b7c 100644
--- a/build/pkgs/gap/spkg-install.in
+++ b/build/pkgs/gap/spkg-install.in
@@ -53,7 +53,7 @@ sdh_install pkg/GAPDoc-* pkg/primgrp-* pkg/SmallGrp-* pkg/transgrp "$GAP_ROOT"/p
 # typically "expected" to be loaded: These are the default packages that are
 # autoloaded at GAP startup (via the PackagesToLoad UserPreference) with an
 # out-of-the-box GAP installation; see
-# https://github.com/sagemath/sage/issues/22626#comment:393 for discussion on this
+# https://github.com/sagemath/sage/issues/22626 for discussion on this
 #
 # Also include atlasrep which is a dependency of tomlib
 sdh_install \
diff --git a/build/pkgs/gdb/distros/conda.txt b/build/pkgs/gdb/distros/conda.txt
index b7a56f9afb3..47326eca441 100644
--- a/build/pkgs/gdb/distros/conda.txt
+++ b/build/pkgs/gdb/distros/conda.txt
@@ -1,2 +1,2 @@
-# Disabled for now because of https://github.com/sagemath/sage/issues/30845#comment:269
+# Disabled for now because of https://github.com/sagemath/sage/issues/30845
 # gdb
diff --git a/build/pkgs/glpk/patches/error_recovery.patch b/build/pkgs/glpk/patches/error_recovery.patch
index a383e25769b..62480b141e5 100644
--- a/build/pkgs/glpk/patches/error_recovery.patch
+++ b/build/pkgs/glpk/patches/error_recovery.patch
@@ -1,6 +1,6 @@
 From: Jeroen Demeyer
 Allow error recovery. See discussion at
-https://github.com/sagemath/sage/issues/20710#comment:18
+https://github.com/sagemath/sage/issues/20710
 
 diff --git a/src/env/error.c b/src/env/error.c
 index a898b76..154de0f 100644
diff --git a/build/pkgs/glpk/spkg-install.in b/build/pkgs/glpk/spkg-install.in
index d1cd0bff9f0..ab7cc208763 100644
--- a/build/pkgs/glpk/spkg-install.in
+++ b/build/pkgs/glpk/spkg-install.in
@@ -1,6 +1,6 @@
 cd src/
 
-# Use newer version of config.guess and config.sub (see Trac #19713)
+# Use newer version of config.guess and config.sub (see Github issue #19713)
 cp "$SAGE_ROOT"/config/config.* .
 
 # Note: The following doesn't work with spaces in `$SAGE_LOCAL`, but we don't
@@ -26,7 +26,7 @@ sdh_configure --with-gmp --disable-static
 sdh_make
 
 # Remove old libraries to make sure we can downgrade it if needed.
-# See https://github.com/sagemath/sage/issues/23596#comment:4 and later.
+# See https://github.com/sagemath/sage/issues/23596.
 rm -f "$SAGE_LOCAL"/lib/libglpk.*
 
 sdh_make_install
diff --git a/build/pkgs/linbox/spkg-install.in b/build/pkgs/linbox/spkg-install.in
index 0f9b25e45d9..6796716374f 100644
--- a/build/pkgs/linbox/spkg-install.in
+++ b/build/pkgs/linbox/spkg-install.in
@@ -24,7 +24,7 @@ fi
 LINBOX_CONFIGURE="--without-fplll $LINBOX_CONFIGURE"
 
 # We disable openmp because of build failures, see
-# https://github.com/sagemath/sage/issues/17635#comment:67
+# https://github.com/sagemath/sage/issues/17635
 # We disable ocl because of build failures, see
 # https://github.com/sagemath/sage/issues/32076
 sdh_configure --with-default="$SAGE_LOCAL" \
diff --git a/build/pkgs/maxima/spkg-install.in b/build/pkgs/maxima/spkg-install.in
index 3ae6382f9ba..2944c869686 100644
--- a/build/pkgs/maxima/spkg-install.in
+++ b/build/pkgs/maxima/spkg-install.in
@@ -1,6 +1,6 @@
 # Sometimes, ECL gives interactive prompts when something goes wrong
 # during the build. Avoid this by redirecting stdin from /dev/null.
-# See https://github.com/sagemath/sage/issues/11884#comment:34
+# See https://github.com/sagemath/sage/issues/11884
 exec </dev/null
 
 
diff --git a/build/pkgs/meataxe/spkg-install.in b/build/pkgs/meataxe/spkg-install.in
index 433e5ba7c86..f34249c2e8d 100644
--- a/build/pkgs/meataxe/spkg-install.in
+++ b/build/pkgs/meataxe/spkg-install.in
@@ -5,7 +5,7 @@
 # marked as installed but the run-time check for the matrix_gfpn_dense
 # module checks whether it can be imported. We need to ensure that these
 # two conditions are equivalent, otherwise strange things can happen.
-# See also https://github.com/sagemath/sage/issues/24359#comment:154
+# See also https://github.com/sagemath/sage/issues/24359
 #
 # This also deletes the static library left behind from the installation
 # of MeatAxe (as opposed to SharedMeatAxe).
diff --git a/build/pkgs/numpy/spkg-install.in b/build/pkgs/numpy/spkg-install.in
index e28660f393f..3683b1f8ae1 100644
--- a/build/pkgs/numpy/spkg-install.in
+++ b/build/pkgs/numpy/spkg-install.in
@@ -7,7 +7,7 @@ if [ `uname` = "Darwin" ]; then
     unset ATLAS
     unset BLAS
     unset LAPACK
-    # https://github.com/sagemath/sage/issues/34110#comment:35
+    # https://github.com/sagemath/sage/issues/34110
     # The fix for "reciprocal" (affected by a clang compiler bug) in
     # https://github.com/numpy/numpy/pull/19926 relies on -ftrapping-math
     # being used when Apple clang v12+ is used.
diff --git a/build/pkgs/ptyprocess/install-requires.txt b/build/pkgs/ptyprocess/install-requires.txt
index a89be777b4a..d20ba760b98 100644
--- a/build/pkgs/ptyprocess/install-requires.txt
+++ b/build/pkgs/ptyprocess/install-requires.txt
@@ -1,3 +1,3 @@
 ptyprocess ==0.5.1
-# https://github.com/sagemath/sage/issues/31280#comment:42 and following
+# https://github.com/sagemath/sage/issues/31280
 # sagelib is not compatible with ptyprocess 0.5.2, 0.6, and 0.7
diff --git a/build/sage_bootstrap/uncompress/tar_file.py b/build/sage_bootstrap/uncompress/tar_file.py
index fdfa8d240e8..f5048afaf9f 100644
--- a/build/sage_bootstrap/uncompress/tar_file.py
+++ b/build/sage_bootstrap/uncompress/tar_file.py
@@ -41,7 +41,7 @@ class SageBaseTarFile(tarfile.TarFile):
     time (the current time), not the timestamps stored in the tarball. This
     is meant to work around https://bugs.python.org/issue32773
 
-    See https://github.com/sagemath/sage/issues/20218#comment:16 and
+    See https://github.com/sagemath/sage/issues/20218 and
     https://github.com/sagemath/sage/issues/24567 for more background.
     """
 
diff --git a/src/sage/arith/constants.pxd b/src/sage/arith/constants.pxd
index 6e5e713707f..3f70243819e 100644
--- a/src/sage/arith/constants.pxd
+++ b/src/sage/arith/constants.pxd
@@ -3,7 +3,7 @@
 # float. We use these instead of decimal constants like 3.1415...
 # because the hex floats are exactly representable as "double", so there
 # shouldn't be any rounding issues in the compiler.
-# See https://github.com/sagemath/sage/issues/23919#comment:15
+# See https://github.com/sagemath/sage/issues/23919
 #
 # Hex floats are standardized in C99, but GCC accepts them
 # unconditionally, also for C++ code.
diff --git a/src/sage/arith/long.pxd b/src/sage/arith/long.pxd
index 4addc8cfc89..ed0704d405f 100644
--- a/src/sage/arith/long.pxd
+++ b/src/sage/arith/long.pxd
@@ -296,7 +296,7 @@ cdef inline bint integer_check_long_py(x, long* value, int* err):
     # - BITS_IN_LONG = 63, PyLong_SHIFT = 30
     # - BITS_IN_LONG = 31, PyLong_SHIFT = 15 (python <= 3.10)
     # - BITS_IN_LONG = 31, PyLong_SHIFT = 30 (new in python 3.11)
-    # cf. https://github.com/sagemath/sage/issues/33842#comment:130
+    # cf. https://github.com/sagemath/sage/issues/33842
     #
     # This way, we know that 1 digit certainly fits in a C long
     # and 4 or more digits never fit.
diff --git a/src/sage/combinat/finite_state_machine.py b/src/sage/combinat/finite_state_machine.py
index 29671fb28ed..b8a7b87c82f 100644
--- a/src/sage/combinat/finite_state_machine.py
+++ b/src/sage/combinat/finite_state_machine.py
@@ -4887,7 +4887,7 @@ def key_function(s):
             return (s.from_state, s.to_state)
         # We use an OrderedDict instead of a dict in order to have a
         # defined ordering of the transitions in the output. See
-        # https://github.com/sagemath/sage/issues/16580#comment:3 . As the
+        # https://github.com/sagemath/sage/issues/16580. As the
         # transitions have to be sorted anyway, the performance
         # penalty should be bearable; nevertheless, this is only
         # required for doctests.
diff --git a/src/sage/cpython/atexit.pyx b/src/sage/cpython/atexit.pyx
index baa3d7a2c0e..d4c806431ea 100644
--- a/src/sage/cpython/atexit.pyx
+++ b/src/sage/cpython/atexit.pyx
@@ -51,7 +51,7 @@ cdef class restore_atexit:
         sage: import atexit
         sage: from sage.cpython.atexit import restore_atexit
         sage: def handler(*args, **kwargs):
-        ....:     import sys # see https://github.com/sagemath/sage/issues/25270#comment:56
+        ....:     import sys # see https://github.com/sagemath/sage/issues/25270
         ....:     sys.stdout.write(str((args, kwargs)))
         ....:     sys.stdout.write('\n')
         sage: atexit.register(handler, 1, 2, c=3)
diff --git a/src/sage/groups/additive_abelian/additive_abelian_wrapper.py b/src/sage/groups/additive_abelian/additive_abelian_wrapper.py
index 3e351585949..570fcc6cc84 100644
--- a/src/sage/groups/additive_abelian/additive_abelian_wrapper.py
+++ b/src/sage/groups/additive_abelian/additive_abelian_wrapper.py
@@ -24,7 +24,7 @@
     (0 : 1 : 0)
     sage: 3000000000000001 * M.0
     (4 : -7 : 1)
-    sage: M == loads(dumps(M))  # known bug, see https://github.com/sagemath/sage/issues/sage_trac/ticket/11599#comment:7
+    sage: M == loads(dumps(M))  # known bug, see https://github.com/sagemath/sage/issues/sage_trac/ticket/11599
     True
 
 TESTS:
diff --git a/src/sage/matrix/matrix_polynomial_dense.pyx b/src/sage/matrix/matrix_polynomial_dense.pyx
index 92502d9632b..6a18cad8644 100644
--- a/src/sage/matrix/matrix_polynomial_dense.pyx
+++ b/src/sage/matrix/matrix_polynomial_dense.pyx
@@ -781,7 +781,7 @@ cdef class Matrix_polynomial_dense(Matrix_generic_dense):
             2021), it would be highly beneficial to use conversions and rely on
             polynomials with matrix coefficients when the matrix size is
             "large" and the degree "small", see
-            :trac:`31472#comment:5`.
+            :trac:`31472`.
         """
         if d <= 0:
             raise ValueError("the precision must be positive")
diff --git a/src/sage/misc/compat.py b/src/sage/misc/compat.py
index d6e5adb431b..b389d793871 100644
--- a/src/sage/misc/compat.py
+++ b/src/sage/misc/compat.py
@@ -49,7 +49,7 @@ def _find_library(name):
 elif sys.platform == 'darwin':
     # On OSX non-standard library paths are not automatically found by the
     # find_library implementation without setting DYLD_LIBRARY_PATH; see
-    # https://github.com/sagemath/sage/issues/21399#comment:25
+    # https://github.com/sagemath/sage/issues/21399
     from ctypes.util import find_library as _orig_find_library
 
     def _find_library(name):
diff --git a/src/sage/schemes/elliptic_curves/ell_field.py b/src/sage/schemes/elliptic_curves/ell_field.py
index d7b6054c150..8edb656b7ae 100644
--- a/src/sage/schemes/elliptic_curves/ell_field.py
+++ b/src/sage/schemes/elliptic_curves/ell_field.py
@@ -988,7 +988,7 @@ def division_field(self, l, names='t', map=False, **kwds):
 
         verbose("Adjoining Y-coordinates of %s-torsion points" % l)
 
-        # THEOREM (Cremona, https://github.com/sagemath/sage/issues/11905#comment:21).
+        # THEOREM (Cremona, https://github.com/sagemath/sage/issues/11905).
         # Let K be a field, E an elliptic curve over K and p an odd
         # prime number. Assume that K contains all roots of the
         # p-division polynomial of E. Then either K contains all
diff --git a/src/sage/schemes/elliptic_curves/ell_torsion.py b/src/sage/schemes/elliptic_curves/ell_torsion.py
index 8a1208569f2..b84212dd104 100644
--- a/src/sage/schemes/elliptic_curves/ell_torsion.py
+++ b/src/sage/schemes/elliptic_curves/ell_torsion.py
@@ -150,7 +150,7 @@ def __init__(self, E):
             sage: type(T)
             <class 'sage.schemes.elliptic_curves.ell_torsion.EllipticCurveTorsionSubgroup_with_category'>
 
-            sage: T == loads(dumps(T))  # known bug, see https://github.com/sagemath/sage/issues/11599#comment:7
+            sage: T == loads(dumps(T))  # known bug, see https://github.com/sagemath/sage/issues/11599
             True
         """
         self.__E = E
diff --git a/src/sage_docbuild/utils.py b/src/sage_docbuild/utils.py
index 196f761650e..8e639774b11 100644
--- a/src/sage_docbuild/utils.py
+++ b/src/sage_docbuild/utils.py
@@ -109,11 +109,11 @@ def build_many(target, args, processes=None):
     least two known issues:
 
     * On versions of Cygwin prior to 3.0.0 there were bugs in mmap handling
-      on threads (see :trac:`27214#comment:25`).
+      on threads (see :trac:`27214`).
 
     * When PARI is built with multi-threading support, forking a Sage
       process from a thread leaves the main Pari interface instance broken
-      (see :trac:`26608#comment:38`).
+      (see :trac:`26608`).
 
     In the future this may be replaced by a generalized version of the more
     robust parallel processing implementation from ``sage.doctest.forker``.