• 
      

    Properly type SCMTool.scmtool_id as a string.

    Review Request #13200 — Created Aug. 10, 2023 and submitted — Latest diff uploaded

    Information

    Review Board
    release-6.x

    Reviewers

    We define SCMTool.scmtool_id as a helper property that computes an ID
    if not already set, based on an Entrypoint. This is here because we used
    to use entrypoint names as the IDs, and while transitioning away from
    that, we needed to maintain compatibility.

    We still need this compatibility shim (whether here or in registration,
    but at the moment it's here). Power Pack still fails to set an SCMTool
    ID for TFS and TFS-Git, so this is still relevant.

    Because we assign this to a property, type checkers think the proper
    type is that of the helper. This change explicitly marks it as a str.

    Unit tests pass.

    Commits

    Files