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-migrate-java
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v2.21.0
Choose a base ref
...
head repository: openrewrite/rewrite-migrate-java
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v2.22.0
Choose a head ref
  • 10 commits
  • 115 files changed
  • 9 contributors

Commits on Aug 2, 2024

  1. Add Refaster style recipes for Guava (#522)

    * Add Refaster style recipes for Guava
    
    * Update src/main/java/org/openrewrite/java/migrate/guava/NoGuavaRefaster.java
    
    Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
    
    ---------
    
    Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
    timtebeek and github-actions[bot] authored Aug 2, 2024
    Configuration menu
    Copy the full SHA
    c1ed883 View commit details
    Browse the repository at this point in the history

Commits on Aug 7, 2024

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

Commits on Aug 8, 2024

  1. Additional No guava recipes (#527)

    * Replace guava TreeMap with Java TreeMap
    Replace guava Primitive asList with Arrays asList
    Replace guava newArrayListWithExpectedSize with Java new ArrayList<>(int)
    
    * Apply org.openrewrite.recipes.OpenRewriteBestPractices
    
    * Remove newArrayListWithExpectedSize to new ArrayList<>(int) as there are nuances lost in this way of doing it
    Remove trailing whitespace
    
    * Also remove newArrayListWithExpectedSize testcase
    Correct more white spacing
    
    Apply feedback to NoGuavaPrimitiveAsList
    
    * Correct more white spacing
    
    * Extend plain JavaIsoVisitor instead of AbstractRefasterJavaVisitor
    
    * Add missing case with Comparator argument
    
    * Reference recipes from NoGuava aggregate
    
    * Update display name for consistency
    
    * Update display name for consistency
    
    ---------
    
    Co-authored-by: Laurens Westerlaken <laurens.westerlaken@jdriven.com>
    Co-authored-by: Tim te Beek <tim@moderne.io>
    3 people authored Aug 8, 2024
    Configuration menu
    Copy the full SHA
    9db7ee7 View commit details
    Browse the repository at this point in the history

Commits on Aug 9, 2024

  1. Clear out empty newlines (#528)

    * Clear out empty newlines
    
    * Add missing document example
    timtebeek authored Aug 9, 2024
    Configuration menu
    Copy the full SHA
    7b360b9 View commit details
    Browse the repository at this point in the history
  2. Recipe ReplaceAWTGetPeerMethod in Java 11 Migration (#524)

    * getPeer() recipe
    
    * updated with fixes
    
    * small updates
    
    * Apply suggestions from code review
    
    Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
    
    * Apply suggestions from code review
    
    * Apply formatter & order imports
    
    * Align before and after code blocks again
    
    * updated recipe name, updated tests and formatted code
    
    * removing extra file commit and test update
    
    * adding comments, Test file cleanup
    
    * Demonstrate the need for `super.visitControlParentheses(cp, ctx)`
    
    * Only override `visitBinary` and `visitInstanceOf`
    
    * Update src/test/java/org/openrewrite/java/migrate/ReplaceAWTGetPeerMethodTest.java
    
    * update to code and adding missing arguments to .yml
    
    * right format for java-version.yml
    
    * Remove import after instanceof replacement
    
    * Also handle flipped case
    
    * Replace `checkClassNameIsEqualToFQCN` with TypeUtils
    
    * Remove duplication in arguments
    
    * Document options and shorten description
    
    * fixed small typo in description
    
    ---------
    
    Co-authored-by: anuram <ranuradh@us.ibm.com>
    Co-authored-by: Tim te Beek <tim@moderne.io>
    Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
    4 people authored Aug 9, 2024
    Configuration menu
    Copy the full SHA
    cdba5a2 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    144a96b View commit details
    Browse the repository at this point in the history
  4. Fix Wro4j tests

    timtebeek committed Aug 9, 2024
    Configuration menu
    Copy the full SHA
    d0375f6 View commit details
    Browse the repository at this point in the history

Commits on Aug 12, 2024

  1. Configuration menu
    Copy the full SHA
    83572ca View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    258b99b View commit details
    Browse the repository at this point in the history
  3. Add Recipe RemovedRuntimeTraceMethods Java 17 (#530)

    * add RemovedRuntimeTraceMethods java 17
    
    * removed duplicate enteries for removeTraceMethods and kept the named Recipe
    
    * End java-version-17.yml with a new line
    
    ---------
    
    Co-authored-by: anuram <ranuradh@us.ibm.com>
    Co-authored-by: Tim te Beek <tim@moderne.io>
    3 people authored Aug 12, 2024
    Configuration menu
    Copy the full SHA
    217cf89 View commit details
    Browse the repository at this point in the history
Loading