Yeikel
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...