Skip to content

chore(route53): typo in route53:useCertificate flag#28755

Closed
juanludlf wants to merge 4 commits intoaws:mainfrom
juanludlf:fix-route53-typo
Closed

chore(route53): typo in route53:useCertificate flag#28755
juanludlf wants to merge 4 commits intoaws:mainfrom
juanludlf:fix-route53-typo

Conversation

@juanludlf
Copy link
Copy Markdown

I noticed this while reading the docs: https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.aws_route53_patterns-readme.html.


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

@aws-cdk-automation aws-cdk-automation requested a review from a team January 18, 2024 13:30
@github-actions github-actions bot added p2 beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK labels Jan 18, 2024
@aws-cdk-automation
Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
  • Commit ID: be80d50
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@aws-cdk-automation aws-cdk-automation added the pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member. label Jan 18, 2024
Copy link
Copy Markdown
Contributor

@kaizencc kaizencc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@juanludlf unfortunately we cannot do this without breaking current users of the typo'd feature flag. Best we can do is create a second feature flag with the correct name, but I don't know if that's worth it at this point

@aws-cdk-automation aws-cdk-automation removed the pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member. label Jan 18, 2024
@kaizencc
Copy link
Copy Markdown
Contributor

kaizencc commented Feb 2, 2024

Closing because the suggestion in this PR cannot be accepted unfortunately.

@kaizencc kaizencc closed this Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK p2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants