• 
      

    Move pytest configuration into pytest.ini.

    Review Request #12264 — Created April 27, 2022 and submitted

    Information

    RBTools
    release-3.x

    Reviewers

    We had placed our pytest configuration in setup.cfg, which is pretty
    common, but deprecated. This isn't really a major problem in practice
    for pytest itself (it's been deprecated for over 5 years yet still
    supported), but the recent addition to the log format configuration
    actually broke building packages on Python 2.7, as the format strings
    could not be expanded to anything.

    There are two places this could conceivably live: pytest.ini or
    pyproject.toml. pyproject.toml is increasingly the future of
    Python package configuration, but unsupported by pytest prior to 6.0,
    which means we can't use it for Python 2.7.

    So this change moves the configuration into pytest.ini instead. This
    is universally supported and has highest precedence.

    Successfully ran pytest on all supported versions of Python.

    Successfully built packages.

    Summary ID
    Move pytest configuration into pytest.ini.
    We had placed our pytest configuration in setup.cfg, which is pretty common, but deprecated. This isn't really a major problem in practice for pytest itself (it's been deprecated for over 5 years yet still supported), but the recent addition to the log format configuration actually broke building packages on Python 2.7, as the format strings could not be expanded to anything. There are two places this could conceivably live: `pytest.ini` or `pyproject.toml`. `pyproject.toml` is increasingly the future of Python package configuration, but unsupported by pytest prior to 6.0, which means we can't use it for Python 2.7. So this change moves the configuration into `pytest.ini` instead. This is universally supported and has highest precedence.
    1858fe435ee4958aa1c3bc485b2490e24831f106
    david
    1. Ship It!
    2. 
        
    chipx86
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-3.x (2deaf47)