Wheels failing to upload with reStructuredText issue.

The wheels are failing to upload with:

Uploading distributions to https://upload.pypi.org/legacy/
Uploading lalsuite-7.15.1.dev20230710-cp310-cp310-macosx_10_15_x86_64.whl
100% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 29.2/29.2 MB • 00:00 • 103.0 MB/s
WARNING  Error during upload. Retry with the --verbose option for more details. 
ERROR    HTTPError: 400 Bad Request from https://upload.pypi.org/legacy/        
         The description failed to render in the default format of              
         reStructuredText. See https://pypi.org/help/#description-content-type  
         for more information.  

I thought that this was fixed by !2173 (merged) but it's still failing?

assigned to @leo-singer

Most recent failure is here

If it's any help, it seems the error was introduced in !2164 (merged)

marked this issue as blocking #691

@david-keitel As @daniel.wysocki points out, this looks like it could be related to !2164 (merged). Any thoughts?

I just ran this through a rst linter, and apparently it's an indentation issue with the bulleted list

>>> import restructuredtext_lint
>>> lint = restructuredtext_lint.lint("""
... LIGO Scientific Collaboration Algorithm Library - minimal Python package
... 
... LALSuite is the LIGO Scientific Collaboration Algorithm Library for
... gravitational-wave analysis. Its primary purpose is searching for and
... characterizing astrophysical signals in gravitational-wave time series data,
... particularly data from ground-based detectors such as `LIGO
... <https://www.ligo.org>`_ and `Virgo <http://www.virgo-gw.eu>`_.
... 
... LALSuite consists of a set of ``configure``-``make``-``install`` style software
... packages organized by problem domain or source classification. This Python
... package provides a standalone, dependency-free binary distribution of the
... libraries and Python modules in LALSuite for Linux and macOS.
... 
... Installing LALSuite from the Python Package Index requires pip >= 19.3.
... 
... To install, simply run::
... 
...     $ pip install lalsuite
... 
... Optional dependencies, which can be installed as ``pip install lalsuite[option]``:
... 
... * ``lalinference`` (pulls in
...   `gwdatafind <https://pypi.org/project/gwdatafind/>`_
...    and `gwpy <https://pypi.org/project/gwpy/>`_)
... * ``lalpulsar`` (pulls in
...   `solar_system_ephemerides <https://pypi.org/project/solar-system-ephemerides/>`_
...   to provide ephemeris files which, until LALSuite 7.15, were still included in
...   the main package)
... * ``test`` (pulls in `pytest <https://pypi.org/project/pytest/>`_)
... """
... )
>>> print(lint)
[<system_message: <paragraph...>>]
>>> print(lint[0])
<system_message level="3" line="25" source="" type="ERROR"><paragraph>Unexpected indentation.</paragraph></system_message>

line 25 is

'* ``lalpulsar`` (pulls in'

Aha, there's an extra space of indentation on the previous line

  `gwdatafind <https://pypi.org/project/gwdatafind/>`_
   and `gwpy <https://pypi.org/project/gwpy/>`_)

and is lined up with gwdatafind, rather than the backtick. Delete that one space and I bet it's fixed

Just submitted a fix in !2175 (merged)

Thanks a lot for fnding that @daniel.wysocki !

I can now see

7.15.1.dev20230711 pre-release about 3 hours ago

so that seems to have indeed fixed it.

closed