dedupe checkpoint version policy formatting

7ed08ed→main·

pfleidi·2w ago·1 file·+2 added/-5 removed

Delegate the empty/default case to formatCheckpointPolicyValue so the version formatter expresses only its one delta, the unsupported case.

Sessions

f54fb5a360a3View transcript

Changes

1

• cmd/entire/cli

  • Mcheckpoint_policy.go+2/-5

113 unmodified lines

}

func formatCheckpointVersionPolicyValue(configured, effective string) string {
    if configured == "" {
        return effective + " (default)"
    }
    if checkpointpolicy.UnsupportedWrite(checkpointpolicy.Policy{
    if configured != "" && checkpointpolicy.UnsupportedWrite(checkpointpolicy.Policy{
        CheckpointVersion:    configured,
        CheckpointMinVersion: checkpointpolicy.DefaultCheckpointVersion(),
    }) {
        return configured + " (unsupported)"
    }
    return configured
    return formatCheckpointPolicyValue(configured, effective)
}

func checkpointPolicyError(message string, err error) error {

Mcmd/entire/cli/checkpoint_policy.go+2/-5