Yeikel

Results 108 comments of Yeikel

> The correct answer is somewhere in the middle... The most "do-no-harm" approach initially would be to check if there is a potential null on the left side and replace...

> @yeikel an example would be a returned value from a non-null API. In that case, the transformation would not be necessary. Thank you Considering what the method does, it...

I understand, thank you for explaining. As a side note, the stream example would need to be refactored to remove the `filter` altogether

> > As a side note, the stream example would need to be refactored to remove the `filter` altogether > > In this case, `find` may be null, so filtering...

@tkvangorder Should we have separate issues for this?

> I think this work will be similar to #1542 > > I would suggest that the takari/maven-wrapper migration be made a separate task. Thank you. I created https://github.com/openrewrite/rewrite/issues/1566

I tested this internally and maven was smart enough to figure out the `distributionUrl` and `wrapperUrl` automatically using my `.m2` settings The issue I found is that it did not...

Should this be Daily or as a standalone check with every PR? (as a set of separate GH action checks) If there are no changes, we shouldn't schedule any run

> These checks often take a lot of time, so running on every PR (change) is probably not a good idea. If we do a side job, we'll need to...

> Hi @yeikel! I think there may be a typo in the `After`. > The map would return the values from `Object#get()`, so the filter can only act on the...