Skip to content

docs: add AWS to the Used by list - #522

Merged
shenxianpeng merged 1 commit into
mainfrom
claude/project-thread-rrzexw
Nov 5, 2026
Merged

shenxianpeng merged 1 commit into
mainfrom
claude/project-thread-rrzexw

Conversation

@shenxianpeng

@shenxianpeng shenxianpeng commented Nov 5, 2026 •

Copy link
Copy Markdown
Member

Adds AWS to the "Used by" list in the README, in the same format as the other organizations.

aws/s2n-tls runs cpp-linter/cpp-linter-action@v2 for clang-tidy in its ci_linting.yml workflow on the default branch.

Summary by CodeRabbit

  • Documentation
    • Added AWS to the “Used by” list.

@shenxianpeng
shenxianpeng requested a review from a team as a code owner November 5, 2026 11:59
@shenxianpeng
shenxianpeng requested review from 2bndy5 and removed request for a team November 5, 2026 11:59
@shenxianpeng shenxianpeng added the documentation Improvements or additions to documentation label Nov 5, 2026 — with Claude
@shenxianpeng shenxianpeng self-assigned this Nov 5, 2026
@coderabbitai

coderabbitai Bot commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The README “Used by” list now includes AWS, with a link to its GitHub organization.

Changes

README listing

Layer / File(s) Summary
Add AWS to the list
README.md
The “Used by” list includes AWS and links to its GitHub organization.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Merge Risk: Minimal · up to a9002

The README-only addition presents no identified merge-blocking risk and is ready to merge after normal checks.

Pre-merge checks: Passed 4

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: adding AWS to the README “Used by” list.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 4ab8e3d9-12a4-4910-8c20-1f52a033bba9
Commits

Reviewing files that changed from the base of the PR and between 5ccb7e8 and a9002b1.

Files selected for processing: 1

  • README.md

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.

@shenxianpeng
shenxianpeng enabled auto-merge (squash) November 5, 2026 12:03

@2bndy5 2bndy5 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I see they are using it, but they seem to be ignoring the clang-tidy findings.

@shenxianpeng
shenxianpeng merged commit b503e19 into main Nov 5, 2026
12 checks passed
@shenxianpeng
shenxianpeng deleted the claude/project-thread-rrzexw branch November 5, 2026 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants