Skip to content

docs(metrics): add documentation for AddDimensions method#1237

Merged
hjgraca merged 1 commit into
aws-powertools:developfrom
lachriz-aws:docs/metrics-util-add-dimensions
Jul 15, 2026
Merged

docs(metrics): add documentation for AddDimensions method#1237
hjgraca merged 1 commit into
aws-powertools:developfrom
lachriz-aws:docs/metrics-util-add-dimensions

Conversation

@lachriz-aws

@lachriz-aws lachriz-aws commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

fixes #1236

Summary

Issue number: #1236

Changes

Adds documentation for the AddDimensions method to the metrics utility docs, alongside the existing AddDimension section. Includes a code snippet, EMF output example, and an info box clarifying that both methods produce the same single dimension set. Also adds a unit test validating the merged behavior and fixes indentation in the existing EMF example.

User experience

Before: Only AddDimension (singular) was documented. Users discovering AddDimensions on the interface had no guidance on usage or behavior.

After: Both methods are documented with examples showing they produce identical EMF output - a single merged dimension set.

Checklist

Please leave checklist items unchecked if they do not apply to your change.

Acknowledgment

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

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

@powertools-for-aws-oss-automation powertools-for-aws-oss-automation Bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jul 14, 2026
@sonarqubecloud

Copy link
Copy Markdown

@boring-cyborg boring-cyborg Bot added documentation Improvements or additions to documentation tests labels Jul 14, 2026

@hjgraca hjgraca left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

GTM

@mergify

mergify Bot commented Jul 15, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@hjgraca
hjgraca merged commit 8ed72cd into aws-powertools:develop Jul 15, 2026
13 checks passed
@codecov

codecov Bot commented Jul 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.46%. Comparing base (0d40609) to head (ed9b2b2).
⚠️ Report is 9 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1237   +/-   ##
========================================
  Coverage    79.46%   79.46%           
========================================
  Files          302      302           
  Lines        12662    12662           
  Branches      1507     1507           
========================================
  Hits         10062    10062           
  Misses        2142     2142           
  Partials       458      458           

☔ View full report in Codecov by Harness.
📢 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

documentation Improvements or additions to documentation size/L Denotes a PR that changes 100-499 lines, ignoring generated files. tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docs: Missing documentation for the AddDimensions method (metrics utility)

2 participants