OK, makes sense https://github.com/apache/groovy/pull/1220
p On Sat, Apr 11, 2020 at 9:13 PM Guillaume Laforge <glafo...@gmail.com> wrote: > No, directly on all the Slurper classes. > Groovy 2.5 and beyond require at least JDK 7, and Path was introduced in > JDK 7. > > Guillaume > > > On Sat, Apr 11, 2020 at 8:57 PM Paolo Di Tommaso < > paolo.ditomm...@gmail.com> wrote: > >> I agree. In which module/class it would be better to add them? groovy-nio >> ? >> >> >> p >> >> On Sat, Apr 11, 2020 at 8:41 PM Guillaume Laforge <glafo...@gmail.com> >> wrote: >> >>> To be consistent, XmlSlurper and YamlSlurper should also get this >>> parse(Path) variant too then :-) >>> That sounds worth a PR! >>> >>> On Sat, Apr 11, 2020 at 7:44 PM Paolo Di Tommaso < >>> paolo.ditomm...@gmail.com> wrote: >>> >>>> Hi all, >>>> >>>> I've just realised that JsonSluper does not have a parse method for a >>>> Path parameter. >>>> >>>> Would you consider a PR to add the support for it? >>>> >>>> >>>> Cheers, >>>> Paolo >>>> >>>> >>> >>> -- >>> Guillaume Laforge >>> Apache Groovy committer >>> Developer Advocate @ Google Cloud Platform >>> >>> Blog: http://glaforge.appspot.com/ >>> Twitter: @glaforge <http://twitter.com/glaforge> >>> >> > > -- > Guillaume Laforge > Apache Groovy committer > Developer Advocate @ Google Cloud Platform > > Blog: http://glaforge.appspot.com/ > Twitter: @glaforge <http://twitter.com/glaforge> >
