Skip to content

Fix(Documentation): Fixed ConfigurationStrSubstitutor dccumenation. #3381

New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom
Merged
merged 1 commit into from
Jan 13, 2025
Merged

Fix(Documentation): Fixed ConfigurationStrSubstitutor dccumenation. #3381

merged 1 commit into from
Jan 13, 2025

Conversation

jaykataria1111
Copy link
Contributor

Fix for #3170

Added a fix for changing ConfigurationStrSubstitutor classes' documentation.

Checklist

  • Base your changes on 2.x branch if you are targeting Log4j 2; use main otherwise NA
  • ./mvnw verify succeeds (if it fails due to code formatting issues reported by Spotless, simply run ./mvnw spotless:apply and retry) NA
  • Trivial change
  • Tests for the changes are provided - NA
  • Commits are signed (optional, but highly recommended)

@jaykataria1111
Copy link
Contributor Author

Hi @ppkarwasz i was able to get this done cause it was a small change

@github-actions GitHub Actions
Copy link

Job Requested goals Build Tool Version Build Outcome Build Scan®
build-macos-latest clean install 3.9.8 ✅ Build Scan PUBLISHED
build-ubuntu-latest clean install 3.9.8 ✅ Build Scan PUBLISHED
build-windows-latest clean install 3.9.8 ❌ Build Scan PUBLISHED
Generated by gradle/develocity-actions

@ppkarwasz ppkarwasz merged commit 5496c72 into apache:2.x Jan 13, 2025
8 of 9 checks passed
@ppkarwasz ppkarwasz mentioned this pull request Jan 13, 2025
Open
26 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jaykataria1111 @ppkarwasz