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
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v8.42.0
Choose a base ref
...
head repository: openrewrite/rewrite
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v8.42.1
Choose a head ref
  • 11 commits
  • 44 files changed
  • 12 contributors

Commits on Dec 18, 2024

  1. Configuration menu
    Copy the full SHA
    2882fdc View commit details
    Browse the repository at this point in the history
  2. fix[maven]: update plugin dependencies (#4798)

    * fix[maven]: update plugin dependencies
    
    * Also change plugin dependencies
    
    ---------
    
    Co-authored-by: Tim te Beek <tim@moderne.io>
    jonblack and timtebeek authored Dec 18, 2024
    Configuration menu
    Copy the full SHA
    6b2b63c View commit details
    Browse the repository at this point in the history

Commits on Dec 19, 2024

  1. Handle erroneous nodes in open rewrite (#4412)

    * Handle erroneous nodes in a tree
    
    * Add visitErroneous to all java parser visitors
    
    * Override the visitVariable to handle erroneous identifier names set by JavacParser
    
    * retain name and suffix for erroneous varDecl
    
    * override the visitVariable to handle error identifiers in all java parser visitors
    
    * Remove sysout
    
    * Update rewrite-java-test/src/test/java/org/openrewrite/java/JavaParserTest.java
    
    Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
    
    * Update rewrite-java-test/src/test/java/org/openrewrite/java/JavaParserTest.java
    
    Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
    
    * handle errors in method params, variable declarations, fix tests
    
    * Add missing license headers
    
    * fix compilation error
    
    * fix compilation error in Java8ParserVisitor
    
    * Apply code suggestions from bot
    
    * fix cases for statementDelim
    
    * fix block statement template generator to handle adding semicolon
    
    * fix ChangeStaticFieldToMethod recipe
    
    * Record compiler errors from erroneous LST nodes
    
    * Adjustments for comments
    
    * Java 17 parser adjustment alos in 8, 11 and 21
    
    * Add `FindCompileErrorsTest` & move away from deprecated `print()`
    
    ---------
    
    Co-authored-by: Jonathan Schnéider <jkschneider@gmail.com>
    Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
    Co-authored-by: Tim te Beek <tim@moderne.io>
    Co-authored-by: aboyko <aboyko@vmware.com>
    5 people authored Dec 19, 2024
    Configuration menu
    Copy the full SHA
    e5d9337 View commit details
    Browse the repository at this point in the history
  2. Make Groovy Parser correctly handle nested parenthesis (#4801)

    * WIP
    
    * Format
    
    * Format
    
    * Move grabbing of whitespace and resetting cursor to where it is actually required
    
    * Extra check is not required
    
    * Use toString
    
    * Add `emptyListLiteralWithParentheses` test
    
    * Add `insideFourParenthesesAndEnters` test
    
    * Move list tests all to ListTest
    
    * Add `emptyMapLiteralWithParentheses`
    
    * Review feedback and fix new testcases
    
    * Add `attributeWithParentheses`
    
    * Improve AttributeTest
    
    * Improve AttributeTest
    
    * Improve AttributeTest
    
    * Improve AttributeTest
    
    * Improve AttributeTest
    
    * Review fix new testcases
    
    * Revert edit to testcase
    
    * Add and fix testcase with newline
    
    * Add JavaDoc and move logic regarding whitespace and resetting cursor
    
    ---------
    
    Co-authored-by: lingenj <jacob.van.lingen@moderne.io>
    Laurens-W and jevanlingen authored Dec 19, 2024
    Configuration menu
    Copy the full SHA
    91a031a View commit details
    Browse the repository at this point in the history

Commits on Dec 20, 2024

  1. suppress javax.json (#4804)

    * suppress javax.json
    
    * Update suppressions.xml
    pstreef authored Dec 20, 2024
    Configuration menu
    Copy the full SHA
    2631a7e View commit details
    Browse the repository at this point in the history
  2. Refactor SpringReference (#4805)

    * Separating and clearer naming
    
    * Add license header
    
    * Review feedback
    Laurens-W authored Dec 20, 2024
    Configuration menu
    Copy the full SHA
    697ae15 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    59273da View commit details
    Browse the repository at this point in the history

Commits on Dec 21, 2024

  1. Add recipe to remove Gradle Enterprise and Develocity (#4809)

    * Add recipe to remove Gradle Enterprise and Develocity
    
    * Remove left over java plugin
    timtebeek authored Dec 21, 2024
    Configuration menu
    Copy the full SHA
    dd2886c View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    e93ec04 View commit details
    Browse the repository at this point in the history

Commits on Dec 23, 2024

  1. Allow file scheme in RemoteArchive to simplify testing (#4791)

    * Allow file scheme in `RemoteArchive` to simplify testing
    
    While it might look a bit controversial, the file scheme can also point to a remote (for instance a mounted network share) file. By allowing the `file://` scheme we can use `RemoteArchive` for those files.
    
    As a useful side effect, this makes testing RemoteArchive handling a lot easier.
    
    * fix test
    
    * Update rewrite-core/src/test/java/org/openrewrite/remote/RemoteArchiveTest.java
    
    Co-authored-by: Sam Snyder <sam@moderne.io>
    
    ---------
    
    Co-authored-by: Sam Snyder <sam@moderne.io>
    pstreef and sambsnyd authored Dec 23, 2024
    Configuration menu
    Copy the full SHA
    da6561e View commit details
    Browse the repository at this point in the history
  2. Try alternative way of determining parenthesis level for `BinaryExpre…

    …ssion` when AST doesn't provide `_INSIDE_PARENTHESES_LEVEL` flag (#4807)
    Laurens-W authored Dec 23, 2024
    Configuration menu
    Copy the full SHA
    e59e48b View commit details
    Browse the repository at this point in the history
Loading