Skip to content

Conversation

@dreamorosi
Copy link
Contributor

@dreamorosi dreamorosi commented Dec 15, 2025

Issue number: closes #7776

Summary

Changes

Please provide a summary of what's being changed

This PR adds a docstring to the BedrockAgentFunctionResolver.__init__ method to document
the serializer parameter, following a similar format and level of detail used in ApiGatewayResolver.

User experience

Please share what the user experience looks like before and after this change


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.

@dreamorosi dreamorosi self-assigned this Dec 15, 2025
@dreamorosi dreamorosi requested a review from a team as a code owner December 15, 2025 15:44
@dreamorosi dreamorosi requested a review from anafalcao December 15, 2025 15:44
@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Dec 15, 2025
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Dec 15, 2025
@codecov
Copy link

codecov bot commented Dec 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.52%. Comparing base (4822e9d) to head (c61fe84).
⚠️ Report is 2 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #7808   +/-   ##
========================================
  Coverage    96.52%   96.52%           
========================================
  Files          275      275           
  Lines        13123    13123           
  Branches       990      990           
========================================
  Hits         12667    12667           
  Misses         353      353           
  Partials       103      103           

☔ 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.

@sonarqubecloud
Copy link

@leandrodamascena leandrodamascena merged commit 0d97c70 into develop Dec 15, 2025
13 checks passed
@leandrodamascena leandrodamascena deleted the docs/serializer_bedrock branch December 15, 2025 16:08
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 event_handlers size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Documentation: add serializer to BedrockAgentFunctionResolver docstring

3 participants