Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: openrewrite/rewrite-logging-frameworks
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v3.9.0
Choose a base ref
...
head repository: openrewrite/rewrite-logging-frameworks
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v3.10.0
Choose a head ref
  • 12 commits
  • 15 files changed
  • 8 contributors

Commits on May 30, 2025

  1. Add recipe to change logger fields to private (#221)

    * Add ChangeLoggersToPrivate recipe
    
    * Add a precondition on using any of the loggers
    
    * Cheap checks first, and use ListUtils to change modifiers
    
    * Include `ChangeLoggersToPrivate` with `Slf4jBestPractices`
    
    * Expect logger to be changed to private in Slf4jBestPracticesTest
    
    * Update src/main/java/org/openrewrite/java/logging/ChangeLoggersToPrivate.java
    
    Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
    
    ---------
    
    Co-authored-by: Tim te Beek <[email protected]>
    Co-authored-by: Tim te Beek <[email protected]>
    Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
    4 people authored May 30, 2025
    Configuration menu
    Copy the full SHA
    14d9d55 View commit details
    Browse the repository at this point in the history

Commits on Jun 2, 2025

  1. Expect renamed variables

    timtebeek committed Jun 2, 2025
    Configuration menu
    Copy the full SHA
    bb60a2b View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    a5431d5 View commit details
    Browse the repository at this point in the history

Commits on Jun 4, 2025

  1. Configuration menu
    Copy the full SHA
    8a30247 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    b8729b7 View commit details
    Browse the repository at this point in the history
  3. Update expectations again

    timtebeek committed Jun 4, 2025
    Configuration menu
    Copy the full SHA
    ad1d202 View commit details
    Browse the repository at this point in the history

Commits on Jun 5, 2025

  1. Configuration menu
    Copy the full SHA
    03807b5 View commit details
    Browse the repository at this point in the history

Commits on Jun 9, 2025

  1. Configuration menu
    Copy the full SHA
    1e14228 View commit details
    Browse the repository at this point in the history

Commits on Jun 16, 2025

  1. Add MatchIsLogLevelEnabledWithLogStatements to SLF4J best practices (

    …#222)
    
    * Add `MatchIsLogLevelEnabledWithLogStatements` to SLF4J best practices
    
    * Do not change conditional for nested catch clause logging statements
    
    * Apply review suggestions
    timtebeek authored Jun 16, 2025
    Configuration menu
    Copy the full SHA
    7839b5d View commit details
    Browse the repository at this point in the history

Commits on Jun 18, 2025

  1. Configuration menu
    Copy the full SHA
    4d92d04 View commit details
    Browse the repository at this point in the history

Commits on Jun 20, 2025

  1. Recipe to remove .toString() called on parameterized logging statem…

    …ent arguments (#224)
    
    Add `StripToStringFromArguments` SLF4j recipe
    ---------
    
    Co-authored-by: Tim te Beek <[email protected]>
    pdelagrave and timtebeek authored Jun 20, 2025
    Configuration menu
    Copy the full SHA
    85518ce View commit details
    Browse the repository at this point in the history

Commits on Jun 23, 2025

  1. fix: match .toString() on any type, not just java.lang.Object (#226)

    `MethodMatcher()` pattern doesn't match subtypes of the specified receiver
    pdelagrave authored Jun 23, 2025
    Configuration menu
    Copy the full SHA
    966498f View commit details
    Browse the repository at this point in the history
Loading