Skip to content

docs: fix "can not" to "cannot" and use HTTPS for external links#37400

Open
syukawa-gh wants to merge 1 commit intoaws:mainfrom
syukawa-gh:fix/misc-comment-improvements
Open

docs: fix "can not" to "cannot" and use HTTPS for external links#37400
syukawa-gh wants to merge 1 commit intoaws:mainfrom
syukawa-gh:fix/misc-comment-improvements

Conversation

@syukawa-gh
Copy link
Copy Markdown
Contributor

Issue

N/A (self-discovered)

Reason for this change

  • "can not" should be "cannot" (single word) per standard English usage in technical documentation
  • External links should use HTTPS

Description of changes

  • aws-cognito/lib/user-pool-client.ts: "can not" → "cannot" in JSDoc (2 occurrences)
  • aws-cognito/lib/user-pool.ts: "can not" → "cannot" in JSDoc
  • aws-s3/lib/bucket.ts: "can not" → "cannot" in comments (2 occurrences)
  • aws-iam/README.md: use HTTPS for OpenID Connect link
  • aws-fsx/README.md: use HTTPS for Lustre website link

Description of how you validated changes

Comment and documentation-only changes. No functional impact.

Checklist

- aws-cognito: "can not" -> "cannot" in JSDoc comments (3 occurrences)
- aws-s3: "can not" -> "cannot" in comments (2 occurrences)
- aws-iam/README.md: use HTTPS for OpenID Connect link
- aws-fsx/README.md: use HTTPS for Lustre website link
@github-actions github-actions bot added the beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK label Mar 30, 2026
@aws-cdk-automation aws-cdk-automation requested a review from a team March 30, 2026 00:27
@github-actions github-actions bot added the p2 label Mar 30, 2026
@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 Mar 30, 2026
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 pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants