• 
      

    Only include package_only_dependencies when building packages.

    Review Request #14865 — Created March 2, 2026 and submitted

    Information

    Review Board
    release-7.1.x

    Reviewers

    The reviewboard build backend writes out a package-dependencies.txt file
    as part of most operations. This was including package_only_dependencies
    all the time, which cause editable installs to fail now that the
    rbintegrations dependency has been bumped to an unreleased dev version.
    This change makes it so when we write out the dependencies file, we only
    include those if we're building a wheel or sdist.

    While in here, I fixed a couple small doc and syntax issues and added
    _write_dependencies() calls to a few additional methods, in case the
    build backend ends up calling them in isolation without first calling
    some of the other methods.

    Did an editable install and saw that it worked.

    Summary ID
    Only include package_only_dependencies when building packages.
    The reviewboard build backend writes out a package-dependencies.txt file as part of most operations. This was including package_only_dependencies all the time, which cause editable installs to fail now that the rbintegrations dependency has been bumped to an unreleased dev version. This change makes it so when we write out the dependencies file, we only include those if we're building a wheel or sdist. While in here, I fixed a couple small docstring issues and added `_write_dependencies()` calls to a few additional methods, in case the build backend ends up calling them in isolation without first calling some of the other methods. Testing Done: Did an editable install and saw that it worked.
    uvmvrpnyvyomnkkymkvstlzxyyyzxspr
    chipx86
    1. Ship It!
    2. 
        
    david
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-7.1.x (ee563c0)