JsonSlurper support for Path

2020-04-11 Thread Paolo Di Tommaso
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

Re: JsonSlurper support for Path

2020-04-11 Thread Guillaume Laforge
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 wrote: > Hi all, > > I've just realised that JsonSluper does not have a parse method for a Path > parameter. > > Would you c

Re: JsonSlurper support for Path

2020-04-11 Thread Paolo Di Tommaso
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 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:

Re: JsonSlurper support for Path

2020-04-11 Thread Guillaume Laforge
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 wrote: > I agree. In which module/class it would be better to add them? groovy-nio > ? > > > p > > On Sat, Apr 11,