Skip to content

Add tagging support for the s3 mb command#10119

Open
aemous wants to merge 2 commits intoaws:developfrom
aemous:v1-s3-tags-mb
Open

Add tagging support for the s3 mb command#10119
aemous wants to merge 2 commits intoaws:developfrom
aemous:v1-s3-tags-mb

Conversation

@aemous
Copy link
Contributor

@aemous aemous commented Mar 11, 2026

v1 port of #10114

Issue #, if available:

Description of changes:

  • Added support for specifying tags on bucket creation for the aws s3 mb command.
  • Added functional tests for testing the new feature.
  • Added a new aws s3 mb command example using the new flag.

Description of tests:

  • Ran all test suites.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@aemous aemous requested review from a team and AndrewAsseily March 11, 2026 01:14
@aemous aemous marked this pull request as ready for review March 11, 2026 01:38
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.41%. Comparing base (c4681b5) to head (d2f0606).
⚠️ Report is 177 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #10119      +/-   ##
===========================================
+ Coverage    93.39%   93.41%   +0.02%     
===========================================
  Files          210      210              
  Lines        17052    17150      +98     
===========================================
+ Hits         15925    16020      +95     
- Misses        1127     1130       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants