Skip to content

Conversation

kylez-ithaka
Copy link
Contributor

@kylez-ithaka kylez-ithaka commented Sep 30, 2025

Summary

The table of environment variables had the wrong variable names, even though other references in the same file had the correct values

Changes

Fixed the names of environment variables for logging

closes #2162

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.

The table of environment variables had the wrong variable names, even though other references in the same file had the correct values
@sonarqubecloud
Copy link

@leandrodamascena
Copy link
Contributor

Hi @kylez-ithaka, thank you so much for submitting this PR! Would you mind opening an issue and linking it to this PR? This way, we can maintain traceability of all the changes we make to the repository.

@kylez-ithaka
Copy link
Contributor Author

@leandrodamascena made and linked an issue

@kylez-ithaka kylez-ithaka removed their assignment Sep 30, 2025
@leandrodamascena leandrodamascena linked an issue Sep 30, 2025 that may be closed by this pull request
1 task
@leandrodamascena
Copy link
Contributor

@leandrodamascena made and linked an issue

Thank you! @phipag will review this soon.

@leandrodamascena leandrodamascena changed the title Fix: Logging environment variables names were incorrect in documentation docs(logger): fix logging environment variables names Sep 30, 2025
Copy link
Contributor

@phipag phipag left a comment

Choose a reason for hiding this comment

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

Thanks @kylez-ithaka for spotting this and sending this PR. LGTM! 🚀

@phipag phipag merged commit e2707c5 into aws-powertools:main Sep 30, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docs: Fix Environment Variables names in table for LOGGER

3 participants