fix: Add me-central-1 support to the client

[stevendotdelahunty]

This change is 

Summary by CodeRabbit

• New Features

  • Added support for the me-central-1 (Middle East Central) AWS region with proper endpoint configuration.

• Tests

  • Added test coverage for the new region's parsing, endpoint resolution, and formatting.

[coderabbitai]

Walkthrough

Added a new AWS region variant MeCentral1 to the Region enum with corresponding implementations for display formatting, string parsing, and endpoint URL resolution, along with test coverage validation.

Changes

Cohort / File(s)	Summary
Region Enum Extension aws-region/src/region.rs	Added MeCentral1 variant to Region enum; updated Display impl to format as "me-central-1"; extended FromStr impl to parse "me-central-1"; added endpoint mapping to "s3.me-central-1.amazonaws.com"; added/updated tests for parsing, endpoint validation, and string representation.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Verify the enum variant naming is consistent with other region identifiers
• Confirm endpoint URL follows AWS naming conventions for the me-central-1 region
• Check that all three trait implementations (Display, FromStr, endpoint) are tested adequately

Poem

🐰 A new region hops into sight,
Me-central-1 burns so bright!
Parsing strings with careful grace,
Endpoints mapped to their place.
From enum variants to tests so keen,
The finest AWS region we've seen! 🌍

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The PR title "Add me-central-1 support to the client" directly and clearly describes the primary change in this pull request. The changeset focuses on adding a new AWS region variant (MeCentral1) to the Region enum and updating all related implementations (Display, FromStr, and endpoint mapping) to support this region. The title is concise, specific, and avoids vague terminology, making it immediately clear to reviewers what the main objective of the PR is.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d88c5f4 and 33d546e.

📒 Files selected for processing (1)

• aws-region/src/region.rs (5 hunks)

🔇 Additional comments (5)

aws-region/src/region.rs (5)

80-81: LGTM! New region variant added correctly.

The MeCentral1 enum variant follows the established pattern and is properly documented.

---

173-173: Display implementation is correct.

The string representation "me-central-1" matches AWS region naming conventions.

---

238-238: String parsing implemented correctly.

The FromStr implementation properly handles "me-central-1" and converts it to the MeCentral1 variant.

---

400-405: Excellent test coverage for the new region.

The test validates both parsing, endpoint resolution, and string representation for the me-central-1 region. This ensures the implementation works correctly end-to-end.

---

296-296: Endpoint format verified as correct.

The endpoint "s3.me-central-1.amazonaws.com" is the correct S3 endpoint for the me-central-1 region.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}