Skip to content

Conversation

kattakaha
Copy link
Contributor

Issue number: #5927

Summary

Changes

This pull request fixes a typo in the code. The word fronzen_openapi_extensions was corrected to frozen_openapi_extensions.

User experience

Before: The typo fronzen_openapi_extensions was present in the code, which might cause confusion.
After: The typo has been corrected to frozen_openapi_extensions, improving code readability and consistency.

Checklist

If your change doesn't seem to apply, please leave them unchecked.

Is this a breaking change?

RFC issue number:

Checklist:

  • Migration process documented
  • Implement warnings (if it can live side by side)

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.

@kattakaha kattakaha requested a review from a team January 27, 2025 20:35
@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jan 27, 2025
Copy link

@leandrodamascena leandrodamascena linked an issue Jan 27, 2025 that may be closed by this pull request
@github-actions github-actions bot added the bug Something isn't working label Jan 27, 2025
@leandrodamascena leandrodamascena changed the title fix(event_handler): frozen openapi extensions option typo #5297 fix(event_handler): fixes typo in variable name fronzen_openapi_extensions Jan 27, 2025
Copy link
Contributor

@leandrodamascena leandrodamascena left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this @kattakaha!! 🚀

APPROVED!

Copy link

codecov bot commented Jan 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.23%. Comparing base (b84a9b2) to head (a313e00).
Report is 2 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #5929   +/-   ##
========================================
  Coverage    96.23%   96.23%           
========================================
  Files          234      234           
  Lines        11010    11010           
  Branches       797      797           
========================================
  Hits         10596    10596           
  Misses         327      327           
  Partials        87       87           

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

@leandrodamascena leandrodamascena merged commit 5548985 into aws-powertools:develop Jan 27, 2025
19 checks passed
Copy link

boring-cyborg bot commented Jan 27, 2025

Awesome work, congrats on your first merged pull request and thank you for helping improve everyone's experience!

@kattakaha kattakaha deleted the fix/typo/frozen#5927 branch January 27, 2025 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 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.

Bug: The fronzen_openapi_extensions word looks like a typo
2 participants