Tim te Beek
Tim te Beek
Thanks a lot for all the hard work here @vudayani ! I've tagged Knut for help with review as I'm traveling, and I know Jonathan has been quite busy these...
> Where are we with this @timtebeek ? What do you think? Is there anything specific missing before this can be merged? Would be awesome to finally push this forward...
Great to hear! I think in the past we'd discussed to maybe add parameters to declarative yaml recipes, which could be a good fit here, although I can't find that...
Thank for the detailed analysis! Here's a reply to your questions at the top: > What do you mean by parameters for declarative recipes. the variables annotated with @Option? So...
> Should I embed them somewhere as documentation or did you think of example implementations, which can directly be used? I think the ready to use preconfigured recipes can go...
Thanks a lot for your thorough work on this one @AlexanderSkrock ; I'd already been following along for a bit in your branch; feel free to open a draft pull...
Quick note here to say I'm traveling this week, so can't fully dive in now. Maybe @kunli2 or @knutwannheden can have a look at fixing your test execution and answering...
Folks subscribed to this issue might be interested to read our latest blog: https://www.moderne.ai/blog/mass-migration-of-nullability-annotations-to-jspecify The recipes we used there are available in OpenRewrite to migrate our internal nullability annotations to...
Thanks for logging an issue @wesboe ! I'm doubting between whether this should be a standalone recipe or whether it should be part of `RemoveRedundantDependencyVersions`. Note that `RemoveRedundantDependencyVersions` could also...
Hi @skazaruddin ; thanks for the detail in your report! I'm seeing your server config `deploymentRepo`, but no corresponding `` referencing that particular server, leading me to doubt if that...