fix(mixins-preview): cannot use string literal types for S3LogsDeliveryProps.permissionsVersion

[mrgrain]

Issue # (if applicable)

@aws-cdk.mixins-preview.aws_logs.S3LogsDelivery-example.ts:19:3 - error TS2322: Type '"permissionsVersion"' is not assignable to type '"V1" | "V2" | undefined'.

19   permissionsVersion: 'permissionsVersion',
     ~~~~~~~~~~~~~~~~~~
@aws-cdk.mixins-preview.aws_logs.S3LogsDeliveryProps-example.ts:16:3 - error TS2322: Type '"permissionsVersion"' is not assignable to type '"V1" | "V2" | undefined'.

16   permissionsVersion: 'permissionsVersion',
     ~~~~~~~~~~~~~~~~~~

  /codebuild/output/src37859859/src/packages/@aws-cdk/mixins-preview/lib/services/aws-logs/logs-delivery.ts:54:12
    54   readonly permissionsVersion?: 'V1' | 'V2';
                  ~~~~~~~~~~~~~~~~~~
    The expected type comes from property 'permissionsVersion' which is declared here on type 'S3LogsDeliveryProps'
[jsii-rosetta] [WARN] 2 diagnostics from assemblies with 'strict' mode on (and 368 more non-strict diagnostics)

Reason for this change

String literal types are not allowed in our pipeline.

Description of changes

Replaced the string literal type with an enum.

Describe any new or updated permissions being added

n/a

Description of how you validated changes

Existing tests.

Checklist

• My code adheres to the CONTRIBUTING GUIDE and DESIGN GUIDELINES

---

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

[aws-cdk-automation]

(This review is outdated)

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

[mergify]

Merge Queue Status

✅ The pull request has been merged

This pull request spent 5 seconds in the queue, with no time waiting for CI.
The checks were run in-place.

Required conditions to merge

• #approved-reviews-by >= 1 [🛡 GitHub branch protection]
  • fix(mixins-preview): cannot use string literal types for S3LogsDeliveryProps.permissionsVersion #36197
• #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
  • fix(mixins-preview): cannot use string literal types for S3LogsDeliveryProps.permissionsVersion #36197
• any of [🛡 GitHub branch protection]:
  • check-success = validate-pr
  • check-neutral = validate-pr
  • check-skipped = validate-pr
• any of [🛡 GitHub branch protection]:
  • check-success = build
  • check-neutral = build
  • check-skipped = build

[mergify]

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

[github-actions]

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.