Sam Snyder

icon indicating a website link https://moderne.io icon indicating an email [email protected]

icon company Moderne Inc

Results 39 comments of Sam Snyder

Unused assignments should be removed

Going to close this for now. While we could hack something together now, a more thorough treatment requires support for dataflow analysis that we haven't yet begun to design.

Function returns should not be invariant

Going to close this for now. While we could hack something together now, a more thorough treatment requires support for dataflow analysis that we haven't yet begun to design.

Assignments should not be redundant

Going to close this for now. While we could hack something together now, a more thorough treatment requires support for dataflow analysis that we haven't yet begun to design.

Maven: Resolve artifactId properties before downloading versions

Thanks for this contribution!

Support Postcondition in DeclarativeRecipe

I share your interest in adding wellness checks to the end of a recipe run and this speaks to that. One I've been thinking about how to implement is validating...

Make ParentPomInsight recursive

This sounds a lot like the existing [ParentPomInsight ](https://github.com/openrewrite/rewrite/blob/a98d83d698dda9971c93a0767b7f85d5866f306d/rewrite-maven/src/main/java/org/openrewrite/maven/search/ParentPomInsight.java#L33)recipe. Can that cover this use-case?

[Gradle] Support updating dependencies in the `dependencyManagement` block

I just merged that linked PR, hopefully that helps

Polish the Gradle build's configuration phase

Thanks @shanman190 !

Implement module imports in Java 25 parser

Since this is a model change be sure to queue up a saas deployment after you merge it