Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

General updates and new supported redaction classes #28

Merged
merged 7 commits into from
Feb 29, 2024

Conversation

GrantBirki
Copy link
Member

@GrantBirki GrantBirki commented Feb 29, 2024

Changes to RedactingLogger:

I tested out GitHub copilot to generate this PR description 🤞

  • lib/redacting_logger.rb: Improved the redacting functionality of the logger by allowing it to handle String, Symbol, and Numeric types, and by using transform_values for Hash types. The shift_size parameter in the initialize method has also been formatted for better readability. [1] [2]

Changes to scripts:

  • script/bootstrap: Added handling for various environment variables to set the APP_ENV value and improved the gem installation process for different environments. A new color code for PURPLE was also added. [1] [2]
  • script/test: Removed unnecessary lines related to bootstrapping and linting.

Changes to tests:

Other changes:

@GrantBirki GrantBirki added enhancement New feature or request dependencies Pull requests that update a dependency file labels Feb 29, 2024
@GrantBirki GrantBirki changed the title General Updates General Updates and new supported redaction classes Feb 29, 2024
@GrantBirki GrantBirki changed the title General Updates and new supported redaction classes General updates and new supported redaction classes Feb 29, 2024
@GrantBirki GrantBirki merged commit d0a7571 into main Feb 29, 2024
4 checks passed
@GrantBirki GrantBirki deleted the general-updates branch February 29, 2024 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants