Properly type SCMTool.scmtool_id as a string.

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

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.

Summary ID
Properly type SCMTool.scmtool_id as a string.
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`.
61c9991d72189a564a76160de5ff4e36bcd415e4
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-6.x (16562c0)
Loading...