Skip to content

UnwrapElseAfterReturn to handle Python source files gracefully#832

Draft
greg-at-moderne wants to merge 3 commits intomainfrom
greg-unwrap-else-python-test
Draft

UnwrapElseAfterReturn to handle Python source files gracefully#832
greg-at-moderne wants to merge 3 commits intomainfrom
greg-unwrap-else-python-test

Conversation

@greg-at-moderne
Copy link
Copy Markdown
Contributor

What's changed?

What's your motivation?

Anything in particular you'd like reviewers to focus on?

Anyone you would like to review specifically?

Have you considered any alternatives or workarounds?

Any additional context

Checklist

  • I've added unit tests to cover both positive and negative cases
  • I've read and applied the recipe conventions and best practices
  • I've used the IntelliJ IDEA auto-formatter on affected files

@greg-at-moderne greg-at-moderne self-assigned this Mar 18, 2026
@github-project-automation github-project-automation bot moved this to In Progress in OpenRewrite Mar 18, 2026
@greg-at-moderne
Copy link
Copy Markdown
Contributor Author

The failure is:

UnwrapElseAfterReturnTest > pythonWithImport() FAILED
    java.lang.IllegalStateException: The Python interpreter at python3 does not have openrewrite 8.76.0.dev20260318050028. Either set pipPackagesPath to allow automatic installation, or install the package manually: pip install openrewrite==8.76.0.dev20260318050028
        at org.openrewrite.python.rpc.PythonRewriteRpc$Builder.get(PythonRewriteRpc.java:550)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

1 participant